Graphviz Issue Tracker
Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002357graphvizBuild/Installpublic2013-09-06 18:302013-09-07 13:18
Reportertk 
Assigned Toellson 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Summary0002357: Unportable test(1) construct in configure script
DescriptionThe configure script has lots of tests using test(1) with the "==" operator.

This comparison operator is only supported by bash; it's not even supported by test from GNU coreutils. Please use the portable "=" comparison operator instead.

If wanted, I can supply a patch. However, in 2.30.0, a simple sed takes care of most cases:
s/" == /" = /

That just leaves one case of s/socklen_t == /socklen_t = /

TagsNo tags attached.
AUXILLARY-FILES
DATE-FIXED
FIX-COMMENTChange applied
FORMER-ID
INPUT-FILE
OUTPUT-FILE
STATUS-COMMENT
VERSION
Attached Files

- Relationships

-  Notes
User avatar (0000499)
ellson (administrator)
2013-09-06 21:33

Personally I prefer "==" as being clearly a test and not an assignment operator.

Also, ksh doesn't complain, so now I'm wondering what else there is to port it to?

- Issue History
Date Modified Username Field Change
2013-09-06 18:30 tk New Issue
2013-09-06 21:33 ellson FIX-COMMENT => Change applied
2013-09-06 21:33 ellson Note Added: 0000499
2013-09-06 21:33 ellson Status new => resolved
2013-09-06 21:33 ellson Resolution open => fixed
2013-09-06 21:33 ellson Assigned To => ellson
2013-09-07 13:18 ellson Status resolved => closed


MantisBT 1.2.5[^]
Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker