Bugzilla Bug 136819

This ebuild represents a stabilized version of avfs which no longer relies on
cvs. This update includes the same features as avfs-0.9.7_p? (namely fuse
support on 2.6.x kernels) without the cvs hassles! Regrettably, I chose to use
_p for the existing ebuild instead of _pre so this will appear as a downgrade,
which it is not. The enhancement is a change in the configure.in script which
compensates for a fuse shortcoming wrt -DFILE_OFFSET_BITS=64 and 64 bit
processors. Bug #133868 highlighted this. So, Stefan, I suppose when you review
and commit, might as well kill the existing avfs-0.9.7_p20060517. I do not have
access to an amd64 machine, but based on the reports from bug #133868, you may
wish to consider adding ~amd64 to the keywords. Jose, please comment!

Created an attachment (id=89207) [details]
avfs-0.9.7.ebuild

Your ebuild installs without problems for me.

A quick test shows that it also works on ~amd64:

$ mountavfs
Mounting AVFS on /home/romildo/.avfs...
$ cd ~/.avfs/usr/local/portage/distfiles/aalib-1.4rc5.tar.gz#/aalib-1.4.0/
$ wc README
  91  543 3604 README
$ cd
$ umountavfs
unMounting AVFS on /home/romildo/.avfs...

cant we just use a standard emake install instead of all the manual doexe'ing?

A few other issues:
- use RDEPEND - the portage version does that already, uh?
- use kernel_is 2 6
- your check in pkg_setup does not work because linux-mod-pkg_setup is not
called
- dont CONFIG_CHECK fuse can be built externally
- make <-> emake, uh?

sorry, this has degraded from portage. Since I believe you also know what is
wrong I think you should put a little more effort in it, please.

(In reply to comment #3)
> cant we just use a standard emake install instead of all the manual doexe'ing?
> 
No. Make install puts lots of unneeded stuff unrelated to fuse support. I'd end
up removing stuff from ${D} manually. There's a comment to that effect in the
ebuild.

> A few other issues:
> - use RDEPEND - the portage version does that already, uh?
> - use kernel_is 2 6

Dangers of cut and paste. I'll have to review...

> - your check in pkg_setup does not work because linux-mod-pkg_setup is not
> called
> - dont CONFIG_CHECK fuse can be built externally
> - make <-> emake, uh?
> 
No. There are two issues. One Fuse support must be enabled in the kernel. Two,
fuse libraries must be installed. Now, I assume you're thinking, well if the
fuse libraries are installed, let it do the kernel check. However, the 2.6
kernel check still has to be done. So relying on fuse installation alone is not
sufficient I think.

> sorry, this has degraded from portage. Since I believe you also know what is
> wrong I think you should put a little more effort in it, please.
> 
I'll take a closer look and see what's changed since the last version of
portage. Since the ebuild is identical with the prior version minus cvs
support, these issues are new to me. You did not raise them before. FWIW
repoman didn't seem to post anything.

Created an attachment (id=89235) [details]
avfs-0.9.7.ebuild

Removes 
- CONFIG_CHECK FUSE_FS since fuse library will install kernel module if not
built into kernel
- direct call to linux-mod-pkg_setup and kernel version check hack
Adds 
- RDEPEND
- kernel_is lt 2 6 (since fuse can compile on 2.4)
Changes
- direct call to chmod to fperms.

Created an attachment (id=89248) [details]
avfs-0.9.7.ebuild

Let's see if three times is the charm.

Changes: streamline pkg_setup, use emake/einstall and remove cruft, cleaned up
whitespaces.

Created an attachment (id=89258) [details]
avfs-0.9.7.ebuild (last one, really!)

corrects whitespace (so much for trying to be neat) and corrects removal of
/etc in ${D}. B/c of einstall, no longer /usr/etc, just /etc. Still needs to
go.

I added this to portage, thanks