First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 142980
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Seemant Kulleen (RETIRED) <seemant@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: James Rowe <jnrowe@ukfsn.org>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
ipkg-utils-missing_deps.patch ipkg-utils-missing_deps.patch patch James Rowe 2006-08-06 07:22 0000 2.22 KB Details | Diff
ipkg-utils-cleanse_useless.patch ipkg-utils-cleanse_useless.patch patch James Rowe 2006-08-06 07:23 0000 2.51 KB Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 142980 depends on: Show dependency tree
Show dependency graph
Bug 142980 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

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







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


Description:   Opened: 2006-08-06 07:21 0000
ipkg-utils fails to declare a couple of dependencies used by the
ipkg-upload script, the first attached patch fixes this.

Thanks


James

------- Comment #1 From James Rowe 2006-08-06 07:22:18 0000 -------
Created an attachment (id=93585) [edit]
ipkg-utils-missing_deps.patch

Seemant,

  I strongly believe the curl and gnupg deps need to be conditional on the
minimal flag, because on many installs where ipkg-utils are useful they are
neither common nor small.  For example, on our n770 images both are within
10% of the size of the GUI web browser package and you wouldn't want to
build packages for uploading on there either :).

  Still learning though, if you think it is inappropriate please kick the
patches back.

Thanks as always


James

------- Comment #2 From James Rowe 2006-08-06 07:23:44 0000 -------
Created an attachment (id=93586) [edit]
ipkg-utils-cleanse_useless.patch

  This second patch reverts my eagerness to use a patch over sed, while
rethinking this it makes a lot more sense to just use a single sed call and
drop the patch.  It is also removing the redundant make install too, I must
have been suffering from some kind of sed reduction induced red mist to have
missed it the first time round.

------- Comment #3 From Jakub Moc 2006-08-06 07:26:27 0000 -------
This should also set RDEPEND explicitely...

------- Comment #4 From Seemant Kulleen (RETIRED) 2006-08-06 07:44:26 0000 -------
Thanks James, in cvs.  We should talk about the mentoring dealy thing stuff
soonish :)

------- Comment #5 From James Rowe 2006-08-06 07:59:07 0000 -------
  Bah, I was just in the middle of writing the comment for the updated patch
wrt to Jakub's comment :)

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