We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 542 - AM_PATH_SDL sets PATH but does not restore original PATH
Summary: AM_PATH_SDL sets PATH but does not restore original PATH
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: build (show other bugs)
Version: 1.2.13
Hardware: Other Solaris
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-24 06:25 UTC by Damien Carbery
Modified: 2008-01-24 07:52 UTC (History)
0 users

See Also:

Attachments
Save old PATH and restore after tests. (462 bytes, patch)
2008-01-24 06:26 UTC, Damien Carbery 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Damien Carbery 2008-01-24 06:25:37 UTC 
In SDL 1.2.13 sdl.m4 the AM_PATH_SDL function looks for sdl-config.
It sets the PATH:
 PATH="$prefix/bin:$prefix/usr/bin:$PATH"
but does not save the current PATH or restore the current PATH at the end.

This breaks the build on Solaris because we have GNU tools in another dir (listed at the top of PATH) but non-GNU tools with the same names in $prefix/bin. Later configure tests finds the non-GNU tools and quits in error.
Comment 1 Damien Carbery 2008-01-24 06:26:16 UTC 
Created attachment 246 [details]
Save old PATH and restore after tests.
Comment 2 Sam Lantinga 2008-01-24 07:52:27 UTC 
This is fixed with subversion revision 3531.

Thanks!