Fix for autoconf misbehaviour caused by cabal #1021

Merged
merged 1 commit into from Oct 18, 2012

Conversation

Projects
None yet
2 participants
Contributor

merijn commented Aug 30, 2012

The autoconf generated configure script tries to automatically determine the
location of the configure script. If the name (argv[0]) of the configure script
does not include a path separator autoconf will first scan the path for a
file named "configure" before defaulting to the current directory.

Cabal previously called configure by doing "sh configure" this will result in
it finding any files named "configure" in the path and using the location of
that file as the basedir to look for (amongst other things) C sources. The
result is that any C sources in the unpacked cabal directories are not found,
resulting in configure/build failures for all such packages (e.g. unix-time)
and all packages depending on those.

Changing cabal to call "sh ./configure" will avoid this behaviour, the presence
of a path separator causes autoconf to default to checking the current
directory first.

@merijn merijn Fixed configure script location detection of autoconf.
The autoconf generated configure script tries to automatically determine the
location of the configure script. If the name (argv[0]) of the configure script
does not include a path separator autoconf will *first* scan the path for a
file named "configure" before defaulting to the current directory.

Cabal previously called configure by doing "sh configure" this will result in
it finding any files named "configure" in the path and using the location of
that file as the basedir to look for (amongst other things) C sources. The
result is that any C sources in the unpacked cabal directories are not found,
resulting in configure/build failures for all such packages (e.g. unix-time)
and all packages depending on those.

Changing cabal to call "sh ./configure" will avoid this behaviour, the presence
of a path separator causes autoconf to default to checking the current
directory *first*.
86c5462
Member

dcoutts commented Sep 19, 2012

Looks ok to me.

@dcoutts dcoutts added a commit that referenced this pull request Oct 18, 2012

@dcoutts dcoutts Merge pull request #1021 from merijn/master
Fix for autoconf misbehaviour caused by cabal
da984e1

@dcoutts dcoutts merged commit da984e1 into haskell:master Oct 18, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment