Opened 18 years ago

Closed 18 years ago

#1868 closed defect (fixed)

pdf lib checks succeed that should fail

Reported by: kyngchaos@… Owned by: mapserverbugs
Priority: high Milestone:
Component: Build Problems Version: 4.8
Severity: normal Keywords:
Cc:

Description

When checking for .so pdf lib, it also tries .sl.  Two dirs that are checked have a logic error that causes 
them to succeed, possibly overriding a previous valid check:

  test -f $PDF_DIR/lib/libpdf.so -o $PDF_DIR/lib/libpdf.sl && PDF_LIBDIR="$PDF_DIR/lib"
  test -f $PDF_DIR/lib64/libpdf.so -o $PDF_DIR/lib64/libpdf.sl && PDF_LIBDIR="$PDF_DIR/lib64"
  test -f $PDF_DIR/libpdf/libpdf.so -o -f $PDF_DIR/libpdf/libpdf.sl && PDF_LIBDIR="$PDF_DIR/libpdf"
  test -f $PDF_DIR/libpdf.so -o -f $PDF_DIR/libpdf.sl && PDF_LIBDIR="$PDF_DIR"

The $PDF_DIR/lib and $PDF_DIR/lib64 are missing a -f flag for the .sl file check.  This block should be:

  test -f $PDF_DIR/lib/libpdf.so -o -f $PDF_DIR/lib/libpdf.sl && PDF_LIBDIR="$PDF_DIR/lib"
  test -f $PDF_DIR/lib64/libpdf.so -o -f $PDF_DIR/lib64/libpdf.sl && PDF_LIBDIR="$PDF_DIR/lib64"
  test -f $PDF_DIR/libpdf/libpdf.so -o -f $PDF_DIR/libpdf/libpdf.sl && PDF_LIBDIR="$PDF_DIR/libpdf"
  test -f $PDF_DIR/libpdf.so -o -f $PDF_DIR/libpdf.sl && PDF_LIBDIR="$PDF_DIR"

Change History (1)

comment:1 by dmorissette, 18 years ago

Resolution: fixed
Status: newclosed
Nice catch. Thanks. I have applied the fix to both 4.10 (will be in
4.10.0-beta2) and the 4.8.x branch (will be in 4.8.5 if we ever release one).
Note: See TracTickets for help on using tickets.