Opened 18 years ago

Closed 18 years ago

Last modified 6 years ago

#8506 closed patch (fixed)

configure prefix problem

Reported by: SF/njoly Owned by: fingolfin
Priority: normal Component: --Other--
Version: Keywords:
Cc: Game:



I just encountered a small prefix configuration problem with CVS scummvm; when `--prefix' and, at least, one `--with-*-prefix' options are set to different path.

Indeed, the configure script, internally use `_prefix' variable for all prefix options. This can lead to an incorrect installation prefix value, as the last defined one will be used:

njoly@cixy [temp/scummvm]> ./configure --prefix=/local --with-mad-prefix=/usr/pkg [...] njoly@cixy [temp/scummvm.ko]> gmake -n install [...] mkdir -p tools/.deps cc -Wall -o tools/md5table tools/md5table.c install -d "/usr/pkg/bin" install -c -s -m 755 "./scummvm" "/usr/pkg/bin/scummvm" install -d "/usr/pkg/man/man6/" install -c -m 644 "./scummvm.6" "/usr/pkg/man/man6/scummvm.6" install -d "/usr/pkg/share/pixmaps/" install -c -m 644 "./scummvm.xpm" "/usr/pkg/share/pixmaps/scummvm.xpm" install -d "/usr/pkg/share/doc/scummvm/" install -c -m 644 "./AUTHORS" "./COPYING" "./NEWS" "./README" "/usr/pkg/share/doc/scummvm/"

I simply fixed this by renaming `_prefix' variable to `__prefix' for all `--with-*-prefix' options.

Hope this helps, Regards.

Ticket imported from: #1411577. Ticket imported from: patches/611.

Attachments (1)

scummvm-cfgprefix.diff (3.2 KB ) - added by SF/njoly 18 years ago.
Fix configure _prefix variable conflict

Download all attachments as: .zip

Change History (4)

by SF/njoly, 18 years ago

Attachment: scummvm-cfgprefix.diff added

Fix configure _prefix variable conflict

comment:1 by fingolfin, 18 years ago

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:2 by fingolfin, 18 years ago

Thanks for pointing this out. I made a different fix, though.

comment:3 by digitall, 6 years ago

Component: --Other--
Summary: configure prefix problem configure prefix problem
Note: See TracTickets for help on using tickets.