ext2_types.h uses conflicting definition of __s64 and __u64 on amd64

Bug #6542 reported by Phillip Susi
4
Affects Status Importance Assigned to Milestone
e2fsprogs (Ubuntu)
Fix Released
Medium
Tollef Fog Heen

Bug Description

The defrag package failed to build from source on amd64 in part because ext2_types.h had a conflicting definition for __s64 and __u64 that differed from those in asm/types.h. This file is in the e2fslibs-dev package which is built from e2fsprogs.

I have corrected the problem by reversing the order that the header file detects 64 bit data types so that it finds long long and uses that instead of long.

Revision history for this message
Phillip Susi (psusi) wrote : Debdiff of fix

.

Phillip Susi (psusi)
Changed in e2fsprogs:
assignee: nobody → tfheen
Revision history for this message
Phillip Susi (psusi) wrote : e2fsprogs_1.38-2ubuntu2.debdiff

Let's try this again. This time I cleaned things up a bit and actually tested that it builds. The original e2fsprogs failed to build from source due to texi2html failing to build some docs. I have no idea how to fix that so I just patched the makefile to ignore errors building and installing the html docs so the package would build.

Tollef Fog Heen (tfheen)
Changed in e2fsprogs:
status: Unconfirmed → Confirmed
Revision history for this message
Theodore Ts'o (tytso) wrote :

Note that a fix to this problem is I believe in 1.39+1.40-WIP-2006.11.14+dfsg-2ubuntu1, which is in both Feisty and Gutsy. So I believe this bug can be marked "Fix Released". Can you test to make sure you are happy now?

Revision history for this message
Theodore Ts'o (tytso) wrote :

As far as I know this bug is already fixed in Gutsy and Fiesty.

Changed in e2fsprogs:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.