First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 66879
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Chris White (RETIRED) <chriswhite@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: mikael söderholm <gnarlie@ihku.org>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
phat-0.2.3.ebuild phat 0.2.3 ebuild application/octet-stream mikael söderholm 2004-10-09 08:31 0000 605 bytes Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 66879 depends on: Show dependency tree
Bug 66879 blocks: 66883
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2004-10-09 08:30 0000
ebuild for phat widgets needed by specimen

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

------- Comment #1 From mikael söderholm 2004-10-09 08:31:22 0000 -------
Created an attachment (id=41425) [details]
phat 0.2.3 ebuild

------- Comment #2 From Chris White (RETIRED) 2004-10-15 18:29:24 0000 -------
Checking this out too.

------- Comment #3 From Chris White (RETIRED) 2004-10-15 18:55:57 0000 -------
Added to cvs :).

A couple o things:

+# $Header: $ 

Need to remember to add the header.  Otherwise there's no
way that repoman will let us commit that.

-KEYWORDS="x86"
+KEYWORDS="~x86"

Initial ebuilds should be ~arch marked until devs can verify that it runs ok.

-IUSE="debug"
+IUSE="debug doc"

I added an extra doc flag explained later.

-        local myconf;
-       use debug || myconf="${myconf} --disable-debug"
-       econf ${myconf} || die
+       econf \
+       $(use_enable debug) \
+       $(use_enable doc gtk-doc) || die
+

There's an easier way than || logic, which is $(use_enable).  Check the 
`man 5 ebuild` docs for information on it.  I also added a doc flag so users
can decide whether or not to install html docs :).

-        make DESTDIR=${D} install || die
-       dodoc AUTHORS COPYING ChangeLog NEWS README TODO
+       make DESTDIR=${D} install || die
+       dodoc AUTHORS ChangeLog NEWS README TODO

removed some whitespace and removed COPYING from docs installed, as it takes
up space and anything that COPYING shows to be impossible to import to the tree
won't get imported anyways :).

Other than that good job :).

First Last Prev Next    No search results available      Search page      Enter new bug