Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 443154 - x11-wm/pekwm-0.1.15 - Doesn't create xsession attributes, new revision for the ebuild proposed.
Summary: x11-wm/pekwm-0.1.15 - Doesn't create xsession attributes, new revision for th...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal with 1 vote (vote)
Assignee: Andreas
URL:
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2012-11-14 20:41 UTC by PhobosK
Modified: 2014-03-10 01:24 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
A new Pekwm 0.1.15 ebuild revision 1, that fixes xsession and contrib problems (pekwm-0.1.15-r1.ebuild,1.75 KB, text/plain)
2012-11-14 20:48 UTC, PhobosK
Details
new_revision.diff (file_443154.txt,2.26 KB, text/plain)
2012-11-14 21:04 UTC, Tom Wijsman (TomWij) (RETIRED)
Details
The new pekwm 0.1.16-r1 (pekwm-0.1.16-r1.ebuild.diff,2.24 KB, text/plain)
2013-01-24 16:31 UTC, PhobosK
Details
The new pekwm 0.1.16-r1 (pekwm-0.1.16-r1.ebuild.diff,2.24 KB, text/plain)
2013-01-24 16:59 UTC, PhobosK
Details

Note You need to log in before you can comment on or make changes to this bug.
Description PhobosK 2012-11-14 20:41:06 UTC
The current x11-wm/pekwm-0.1.15 has two problems:
1. It doesn't create/install an xsession attributes needed for GDM/KDM and simple XSession to sense it and show it as an option in their UI.
2. There are two user contributed perl scripts (in contrib/lobo) that are not installed at all. According to upstream, they should go to the final doc folder.

I am proposing a new revision of the ebuild (x11-wm/pekwm-0.1.15-r1) which fixes these problems.

Reproducible: Always
Comment 1 PhobosK 2012-11-14 20:48:15 UTC
Created attachment 329560 [details]
A new Pekwm 0.1.15 ebuild revision 1, that fixes xsession and contrib problems

Changes:
1. Added "contrib" IUSE flag that controls if the contrib perl scripts from "contrib/lobo" will be installed.
2. Added deps of perl and zenity for the "contrib" flag, since the upstream perl scripts need them
3. Added postinst message to the user about the contrib scripts
4. Added xsession files install for KDM/GDM and XSession
5. Changed description of the application to: "A lightweight window mananger initially based on aewm++"
Comment 2 PhobosK 2012-11-14 20:51:44 UTC
Since it seems @theodor_ AT _suremail.info often version bumps pekwm, I am putting him to CC, to inform him too.
Comment 3 Tom Wijsman (TomWij) (RETIRED) gentoo-dev 2012-11-14 21:04:07 UTC
Created attachment 329564 [details]
new_revision.diff

Diff between pekwm-0.1.15 and new revision
Comment 4 theodor 2013-01-23 20:17:11 UTC
Thanks for adding me.

I only have minor point, shouldn't the two elogs at the end depend on the contrib USE flag?
Comment 5 PhobosK 2013-01-24 16:31:20 UTC
Created attachment 336732 [details]
The new pekwm 0.1.16-r1

Yeah @Theodor, you are right... it should depend... my mistake sorry....

Meanwhile there was a new upstream version of 0.1.16, and it seems that @Jeroen Roovers just bumped the old ebuild just changing it to EAPI 5, with none of my proposed changes...

So I am uploading the new 0.1.16 ebuild as a new revision... I hope it will end up in the portage tree before a new 0.1.17 upstream version appears ;) :)
Comment 6 PhobosK 2013-01-24 16:59:23 UTC
Created attachment 336734 [details]
The new pekwm 0.1.16-r1

There was an initial typo in the description of the package that i didn't see (the "mananger" word). Fixed it.
Comment 7 Jeroen Roovers (RETIRED) gentoo-dev 2014-03-10 01:24:34 UTC
Fixed in 0.1.17 as per bug #471340.