aMule Bug Tracker - aMule
View Issue Details
0001572aMuleMulti Platformpublic2009-12-30 13:112014-09-12 09:55
0001572: [Debian] failed to build on alpha and hppa - wrong fallocate check

aMule 2.2.6 fails to build on alpha and hppa.

Full log: [^]

I had a quick look at config.log, and it seems that fallocate() in alpha works with gcc and doesn't work with g++. Should fallocate() be checked with g++?

No tags attached.
patch fallocate-dev.patch (710) 2010-05-19 21:07
patch fallocate-2.2.6.patch (662) 2010-05-19 21:07
Issue History
2009-12-30 13:11DerevkoNew Issue
2009-12-30 13:11DerevkoOperating System => Debian
2010-05-19 10:04GonoszTopiNote Added: 0003438
2010-05-19 10:04GonoszTopiAssigned To => GonoszTopi
2010-05-19 10:04GonoszTopiStatusnew => assigned
2010-05-19 10:04GonoszTopiStatusassigned => feedback
2010-05-19 10:23DerevkoNote Added: 0003439
2010-05-19 21:07GonoszTopiFile Added: fallocate-dev.patch
2010-05-19 21:07GonoszTopiFile Added: fallocate-2.2.6.patch
2010-05-19 21:10GonoszTopiNote Added: 0003440
2014-09-12 09:55GonoszTopiFixed in Revision => 10838
2014-09-12 09:55GonoszTopiNote Added: 0003637
2014-09-12 09:55GonoszTopiStatusfeedback => resolved
2014-09-12 09:55GonoszTopiResolutionopen => fixed
2014-09-12 09:55GonoszTopiFixed in Version => 2.3.2

2010-05-19 10:04   
This seems to be a gcc (or installed headers/libraries) bug, then. Though, your suggestion also makes sense: we use g++ almost always, but for the checks we use gcc (except where a C++ compiler is explicitly needed). Thus the inconsistency between the usage and the test case is real.

However, since I don't have access to an alpha I can't test whether fixing this issue really fixes alpha build (i.e. there are no other functions with the same problem). If you can and will test such a patch, I'll be glad to sort this thing out.
2010-05-19 10:23   
which patch do you refer?

In the Debian package I disabled fallocate() in alpha and hppa: [^]
2010-05-19 21:10   
I referred to a theoretical patch that just came into existance. ;)

I made two versions of the patch, one for the 2.2.6 release, the other for a current development snapshot. Please note, that the patch only modifies the autoconf input file, thus you'll need to run ./ (or autoconf at least) after applying.
2014-09-12 09:55   
Applied the patch, and more. Hopefully it doesn't break anything else.