Discussion:
Bug#725218: sed: FTBFS, missing PATH_MAX on hurd
(too old to reply)
Paolo Bonzini
2013-10-03 16:14:11 UTC
Permalink
Package: sed
Version: 4.2.2-2
Severity: important
Tags: patch
Usertags: hurd
Hello,
hurd doesn't define PATH_MAX and upstream decided not to set it themself anymore
since 4.2.2. See http://sources.debian.net/src/sed/4.2.2-2/lib/pathmax.h
That's a gnulib change.
Attached patch01 sets PATH_MAX in d/rules.
Alternatively patch02 defines it in pathmax.h above.
Thanks for considering.
PATH_MAX is not really needed, it can be replaced by any constant
because sed anyway does reallocation if the buffer needs to be larger
than PATH_MAX.

Jose, perhaps you can change it to 200 and drop the pathmax module from
bootstrap.conf?

Paolo
Jose E. Marchesi
2013-10-06 12:45:47 UTC
Permalink
Attached patch01 sets PATH_MAX in d/rules.
Alternatively patch02 defines it in pathmax.h above.
Thanks for considering.
PATH_MAX is not really needed, it can be replaced by any constant
because sed anyway does reallocation if the buffer needs to be larger
than PATH_MAX.

Jose, perhaps you can change it to 200 and drop the pathmax module from
bootstrap.conf?

Done. PATH_MAX is now defined in basicdefs.h in case it is not provided
externally.
Samuel Thibault
2014-02-09 17:11:48 UTC
Permalink
Hello,

Here is a backport of the fix for the debian package, could you upload
it?

Thanks,
Samuel

Loading...