Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 123253 - Cannot build findutils-4.3.0 on Gentoo/NetBSD
Summary: Cannot build findutils-4.3.0 on Gentoo/NetBSD
Status: RESOLVED FIXED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: Other (show other bugs)
Hardware: All Other
: High normal (vote)
Assignee: Gentoo/BSD Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-18 04:41 UTC by Benigno B. Junior (RETIRED)
Modified: 2006-09-16 19:40 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
findutils-4.3.0-bsd.patch (findutils-4.3.0-bsd.patch,794 bytes, patch)
2006-02-18 04:43 UTC, Benigno B. Junior (RETIRED)
Details | Diff
findutils-4.3.0.ebuild-bsd.patch (findutils-4.3.0.ebuild-bsd.patch,430 bytes, patch)
2006-02-18 04:44 UTC, Benigno B. Junior (RETIRED)
Details | Diff
Ebuild patch (findutils-ebuild.patch,1.35 KB, patch)
2006-02-22 03:59 UTC, Diego Elio Pettenò (RETIRED)
Details | Diff
Ebuild patch (findutils-ebuild.patch,1.37 KB, patch)
2006-02-22 04:14 UTC, Diego Elio Pettenò (RETIRED)
Details | Diff
findutils-4.3.0-statvfs-hack.patch (findutils-4.3.0-statvfs-hack.patch,2.99 KB, patch)
2006-02-22 16:07 UTC, Benigno B. Junior (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Benigno B. Junior (RETIRED) gentoo-dev 2006-02-18 04:41:22 UTC
I've been searching for statfs inside NetBSD's headers and it seems to be changed to statvfs, but findutils doesn't see this in it's configuration step.

So I'm attaching a working ebuild and patch for Gentoo/NetBSD and Linux (I tested it on Linux, and there's no inteference). We have no problem like this bug #114747 and we cannot use our own regex.h, so the patch for the ebuild is needed. I don't know about this statfs on others systems, but it might work on some. Changing it to fit others BSD's might be easy.

I'm going to attach the things in the next posts.
Comment 1 Benigno B. Junior (RETIRED) gentoo-dev 2006-02-18 04:43:01 UTC
Created attachment 80090 [details, diff]
findutils-4.3.0-bsd.patch

Here's the patch for the sources.
Comment 2 Benigno B. Junior (RETIRED) gentoo-dev 2006-02-18 04:44:06 UTC
Created attachment 80091 [details, diff]
findutils-4.3.0.ebuild-bsd.patch

Here's the patch for the ebuild.
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-02-18 05:37:33 UTC
Need to check this a bit more, findutils 4.3.0 still doesn't work on either fbsd and dfly with this patch.

I tried also to update gnulib's support but it's a bit messed up.
I think we'll need to wait for findutils 4.3.1 hoping they'll update gnulib..
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-02-22 03:41:14 UTC
Benigno, did you check if newer gnulib has that problem fixed?
Comment 5 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-02-22 03:58:34 UTC
Okay seems like the use of external regex breaks fbsd badly.
I'm going to attach the an updated ebuild patch that enables the internal regex when the libc does not provide the one used.

The nbsd patch is sort of a problem, it breaks at least FreeBSD, so it has to be fixed and generalised and sent to gnulib maintainers (not to findutils maintainer that are just using gnulib), when that is done, findutils maintainers should be told to update the gnulib they use so that fixed version can be pulled in.
Comment 6 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-02-22 03:59:35 UTC
Created attachment 80422 [details, diff]
Ebuild patch
Comment 7 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-02-22 04:14:04 UTC
Created attachment 80424 [details, diff]
Ebuild patch

Forgot ${myconf}.
Comment 8 Benigno B. Junior (RETIRED) gentoo-dev 2006-02-22 16:06:36 UTC
(In reply to comment #4)
> Benigno, did you check if newer gnulib has that problem fixed?
> 

I only found fsusage.m4 and its corresponding source fsusage.c, which seems to use statvfs, but importing it is a bit ridiculous (50kb+) and will not do what we need without changing mountlist.c. I'll send an email to findutils' upstream about this.

So I created an ugly hack (I know it might be stupid for upstream, though), which only affect the system that has no statfs.h but has statvfs.h (the case of NetBSD 2+. Is there other in the way?). Sir reb, do you need this?

I used the eautoconf to regenerate the configure script. I'm attaching the patch in the next post. If someone knows how to correct this in a pretty way, please, don't hesitate in helping! This is what I could do with minimal changes. :)
Comment 9 Benigno B. Junior (RETIRED) gentoo-dev 2006-02-22 16:07:52 UTC
Created attachment 80484 [details, diff]
findutils-4.3.0-statvfs-hack.patch

It's there.
Comment 10 SpanKY gentoo-dev 2006-04-10 19:56:18 UTC
so do we have a clean fix here ?
Comment 11 Benigno B. Junior (RETIRED) gentoo-dev 2006-04-10 20:35:58 UTC
No. I've sent some emails to gnulib's upstream, but didn't get them back. Newers coreutils have the same problem. Let's just schedule this for later, it's not really mandatory now, since 4.1 still works.. :)
Comment 12 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-04-18 11:43:39 UTC
As Mike broke also the working findutils...
Comment 13 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-04-18 12:10:18 UTC
Is it an hack? Yes.
Is it ugly? Yes.

Did you break the only working version? Yes.

So thank you all for playing the ugly hack game :P


Serious bit: as this was broken, and the fix for glibc breaks the rest, I consider this whole mess unfixable in other ways until upstream reacts, so I added the needed fixes anyway.
Mike please next time you do something that you know breaks for us, do it in a revbump and drop the keyword. Thanks.
Comment 14 SpanKY gentoo-dev 2006-04-18 15:30:26 UTC
the bug stays open until the hack is gone from portage
Comment 15 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-04-18 15:34:31 UTC
okay then.
Comment 16 SpanKY gentoo-dev 2006-09-16 19:40:39 UTC
4.3.1 should work properly