[gentoo-dev] repoman ebuild changes

[gentoo-dev] repoman ebuild changes

[Michael Sterrett -Mr. Bones.-]

Greetings -

    Occasionally, I've contacted developers about repoman issues with
    ebuilds when things are broken in portage.  I generally run repoman
    on the entire portage tree - a process that takes quite some time -
    since not everyone wants to do this.

    Frequently, the only new changes in the repoman report are the addition
    of some warnings regarding whitespace - either Leading or Trailing.
    For these warnings, I generally just fix the ebuilds myself since the
    change is obvious and contacting developers about whitespace warnings
    is pretty silly.  If you find this happening to your favorite ebuild
    please just take this as a gentle nudge to not ignore the repoman
    warnings when you commit and move on with your life.

    My expectation is that every commit to the portage tree is as
    "repoman free" as possible.  At one point, the repoman report was
    over 20k lines.  Let's not go there again.

Thanks,

Michael Sterrett
  -Mr. Bones.-
mr_bones_@gentoo.org


--
gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] repoman ebuild changes

[Aron Griffis]

[-- Attachment #1: Type: text/plain, Size: 738 bytes --]

Speaking of repoman, I'd like to add a couple more checks to it:

1. All variables listed in IUSE are mentioned in one of the src_*
   functions (e.g. "use_with gtk").  It isn't enough to put them in
   SRC_URI, because that means that the build will be different between
   machines, even with the same USE flags.

2. ${D} and ${S} (and possibly others) can't be used in pkg_* functions
   because those dirs don't exist when installing from a binary package.

Regarding #1, I plan to spam bugzilla soon with a large number of bugs
to get the existing ebuilds cleaned up.

Aron

-- 
Aron Griffis
Gentoo Linux Developer (alpha / ia64 / ruby / vim)
Key fingerprint = E3B6 8734 C2D6 B5E5 AE76  FB3A 26B1 C5E3 2010 4EB0


[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]