Bug 384569 – New ebuild: kfaenza-0.8.8.ebuild

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Shubhadeep Chaudhuri 2011-09-26 23:45:56 UTC]

Created attachment 287843 [details]
Ebuild for version 0.8.5

Hi! This is a KDE port of the faenza iconpack and also has many other iconpacks integrated into it. It's a very comprehensive icon set for KDE. The preview can be
found here http://kde-look.org/content/show.php?content=143890

It should probably be put in as x11-themes/oxygenkde.

[Comment 1 Shubhadeep Chaudhuri 2011-09-26 23:50:14 UTC]

(In reply to comment #0)
> It should probably be put in as x11-themes/oxygenkde.

Should be put in as x11-themes/kfaenza. Sorry, i just pasted it from another ebuild inclusion bug i submitted a while ago.

[Comment 2 Shubhadeep Chaudhuri 2011-09-27 12:53:56 UTC]

Created attachment 287925 [details]
Updated ebuild for version 0.8.5

[Comment 3 Markos Chandras (RETIRED) 2011-09-28 00:22:25 UTC]

Hello, The Gentoo Team would like to firstly thank you for your ebuild 
submission. We also apologize for not being able to accommodate you in a timely
manner. There are simply too many new packages.

Allow me to use this opportunity to introduce you to Gentoo Sunrise. The 
sunrise overlay[1] is a overlay for Gentoo which we allow trusted users to 
commit to and all users can have ebuilds reviewed by Gentoo devs for entry 
into the overlay. So, the sunrise team is suggesting that you look into this 
and submit your ebuild to the overlay where even *you* can commit to. =)

Thanks,
On behalf of the Gentoo Sunrise Team,
Markos.

[1]: http://www.gentoo.org/proj/en/sunrise/
[2]: http://overlays.gentoo.org/proj/sunrise/wiki/SunriseFaq

[Comment 4 Shubhadeep Chaudhuri 2011-09-28 17:37:00 UTC]

Created attachment 288103 [details]
ebuild for version 0.8.5

[Comment 5 Adam 2011-10-15 00:03:29 UTC]

Created attachment 289841 [details]
KFaenza-0.8.7.ebuild

Changes:

New source:
SRC_URI="http://ompldr.org/vYXB0aw/${PN}-${PV}.tar.gz"

--------------------------------

I decided to replace

src_install() {
	insinto /usr/share/icons
	doins -r KFaenza || die
}

with

src_install() {
	dodir /usr/share/icons
	cp -R "${S}/" "${D}"/usr/share/icons
}

The first method needs much more time and ressources (especially on my laptop)

time with doins and instinto:

real    4m45.424s
user    2m3.135s
sys     1m47.553s

time with dodir and cp:

real    1m42.575s
user    1m25.734s
sys     0m17.659s

I hope it's o.k.

[Comment 6 Adam 2011-10-15 08:35:54 UTC]

Created attachment 289851 [details]
kfaenza-0.8.7.ebuild corr.

[Comment 7 Adam 2011-10-15 11:44:26 UTC]

Created attachment 289869 [details]
kfaenza-0.8.7.ebuild

corr.:

dodir /usr/share/icons || die

[Comment 8 Adam 2011-10-15 19:24:00 UTC]

Created attachment 289899 [details]
corr. download link

[Comment 9 Shubhadeep Chaudhuri 2011-10-16 01:51:20 UTC]

great. nice of u to do benchmarks. but remove the outer loop. its not needed. put back IUSE. and should be $Header: $
the gentoo ebuild howto says so.
the last corrcted ebuild doesn't have || die. and remove the restrict.

[Comment 10 Adam 2011-10-16 03:42:18 UTC]

(In reply to comment #9)
> great. nice of u to do benchmarks. but remove the outer loop. its not needed.
> put back IUSE. and should be $Header: $
> the gentoo ebuild howto says so.
> the last corrcted ebuild doesn't have || die. and remove the restrict.

Hi!

Thank you for the correction.
The ebuild is in tree.
Feel free to post new attachments.
Have a nice day!

[Comment 11 Shubhadeep Chaudhuri 2011-10-22 12:49:12 UTC]

Created attachment 290511 [details]
kfaenza-0.8.8.ebuild

Ebuild for version 0.8.8

[Comment 12 Shubhadeep Chaudhuri 2011-10-22 12:51:18 UTC]

Created attachment 290513 [details]
kfaenza-0.8.8.ebuild

Ebuild for version 0.8.8

[Comment 13 Shubhadeep Chaudhuri 2011-11-01 18:34:34 UTC]

please mark fixed