My View | View Issues | Change Log | Roadmap
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001572aMuleMulti Platformpublic2009-12-30 13:112014-09-12 09:55
ReporterDerevko 
Assigned ToGonoszTopi 
PrioritynormalSeveritymajorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version2.2.6 
Target VersionFixed in Version2.3.2 
Summary0001572: [Debian] failed to build on alpha and hppa - wrong fallocate check
DescriptionHi,

aMule 2.2.6 fails to build on alpha and hppa.

Full log: https://buildd.debian.org/fetch.cgi?pkg=amule&arch=alpha&ver=2.2.6%2Bdebian0-2&stamp=1262114500&file=log&as=raw [^]

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++?

Cheers,
Giuseppe.
TagsNo tags attached.
Fixed in Revision10838
Operating SystemDebian
Attached Filespatch file icon fallocate-dev.patch [^] (710 bytes) 2010-05-19 21:07 [Show Content]
patch file icon fallocate-2.2.6.patch [^] (662 bytes) 2010-05-19 21:07 [Show Content]

- Relationships

-  Notes
(0003438)
GonoszTopi (administrator)
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.
(0003439)
Derevko (reporter)
2010-05-19 10:23

 which patch do you refer?

In the Debian package I disabled fallocate() in alpha and hppa:

http://patch-tracker.debian.org/patch/series/view/amule/2.2.6+debian0-7/fallocate.diff [^]
(0003440)
GonoszTopi (administrator)
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 configure.in, thus you'll need to run ./autogen.sh (or autoconf at least) after applying.
(0003637)
GonoszTopi (administrator)
2014-09-12 09:55

 Applied the patch, and more. Hopefully it doesn't break anything else.

- Issue History
Date Modified Username Field Change
2009-12-30 13:11 Derevko New Issue
2009-12-30 13:11 Derevko Operating System => Debian
2010-05-19 10:04 GonoszTopi Note Added: 0003438
2010-05-19 10:04 GonoszTopi Assigned To => GonoszTopi
2010-05-19 10:04 GonoszTopi Status new => assigned
2010-05-19 10:04 GonoszTopi Status assigned => feedback
2010-05-19 10:23 Derevko Note Added: 0003439
2010-05-19 21:07 GonoszTopi File Added: fallocate-dev.patch
2010-05-19 21:07 GonoszTopi File Added: fallocate-2.2.6.patch
2010-05-19 21:10 GonoszTopi Note Added: 0003440
2014-09-12 09:55 GonoszTopi Fixed in Revision => 10838
2014-09-12 09:55 GonoszTopi Note Added: 0003637
2014-09-12 09:55 GonoszTopi Status feedback => resolved
2014-09-12 09:55 GonoszTopi Resolution open => fixed
2014-09-12 09:55 GonoszTopi Fixed in Version => 2.3.2

Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker