Bug 433718 – Version bumps: app-text/zathura-0.2.1, app-text/zathura-cb-0.1.0-r1, app-text/zathura-djvu-0.2.1, app-text/zathura-pdf-poppler-0.2.1, app-text/zathura-ps-0.2.1, dev-libs/girara-0.1.4

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Simon Kohlmeyer 2012-09-02 21:53:06 UTC]

Created attachment 322778 [details]
app-text/zathura-0.2.1.ebuild

My aim was an upgrade of zathura to 0.2.1, but I think it's always wise to keep plugins up to date. dev-libs/girara is a dependency of zathura by the same developers, and girara-0.1.4 is required by zathura-0.2.1. Enclosed are ebuilds for the following packages:

app-text/zathura-0.2.1
app-text/zathura-pdf-poppler-0.2.1
app-text/zathura-djvu-0.2.1
dev-libs/girara-0.1.4
app-text/zathura-cb-0.1.0-r1
app-text/zathura-ps-0.2.0-r1

These new ebuilds include a few changes. I assumed dev-libs/girara-1.3 is in slot 2 because of historical reasons, so I put the new version in slot 0.

This forced me to update all packages depending on girara, which is the reason for the last two ebuilds. The only change to them is in the dependency for girara in slot 0. The first four are regular updates.

PS: I only have limited experience contributing ebuilds. If you think somethings wrong, it probably is.

[Comment 1 Simon Kohlmeyer 2012-09-02 21:54:48 UTC]

Created attachment 322780 [details]
app-text/zathura-cb-0.1.0-r1.ebuild

[Comment 2 Simon Kohlmeyer 2012-09-02 21:55:12 UTC]

Created attachment 322782 [details]
app-text/zathura-djvu-0.2.1.ebuild

[Comment 3 Simon Kohlmeyer 2012-09-02 21:55:49 UTC]

Created attachment 322784 [details]
app-text/zathura-pdf-poppler-0.2.1.ebuild

[Comment 4 Simon Kohlmeyer 2012-09-02 21:56:15 UTC]

Created attachment 322786 [details]
app-text/zathura-ps-0.2.0-r1.ebuild

[Comment 5 Simon Kohlmeyer 2012-09-02 21:56:44 UTC]

Created attachment 322788 [details]
dev-libs/girara-0.1.4.ebuild

[Comment 6 Sergey Popov 2012-09-03 10:13:40 UTC]

Confirming and adding packages maintainers to CC. Next time, please provide diffs rather then full ebuilds in case, when changes are minimal.

[Comment 7 Simon Kohlmeyer 2012-09-03 12:37:43 UTC]

Thanks for the hint, I'll do that from now on.

[Comment 8 Simon Kohlmeyer 2012-09-03 16:52:11 UTC]

Just noticed that this installs an empty man1 file for app-text/zathura. Investigating...

[Comment 9 Samuli Suominen (RETIRED) 2012-09-03 18:35:25 UTC]

Comment on attachment 322788 [details]
dev-libs/girara-0.1.4.ebuild

The src_prepare() was outdated in this one. Correct version in Portage now.

[Comment 10 Samuli Suominen (RETIRED) 2012-09-03 18:52:08 UTC]

Comment on attachment 322780 [details]
app-text/zathura-cb-0.1.0-r1.ebuild

Incorrect removal of SLOT dep of girara,

@@ -15,7 +15,7 @@
 IUSE=""
 
 COMMON_DEPEND=">=app-text/zathura-0.2.0
-	>=dev-libs/girara-0.1.3
+	>=dev-libs/girara-0.1.3:2

And pointless revision bump, deleting attachment

[Comment 11 Samuli Suominen (RETIRED) 2012-09-03 18:53:01 UTC]

Comment on attachment 322786 [details]
app-text/zathura-ps-0.2.0-r1.ebuild

Same problem as with previous one, incorrect removal of SLOT depend for girara

Also pointless revbump

[Comment 12 Samuli Suominen (RETIRED) 2012-09-03 18:53:31 UTC]

New girara, zathura, zathura-pdf-poppler and zathura-djvu in Portage

[Comment 13 Samuli Suominen (RETIRED) 2012-09-03 18:54:18 UTC]

Next time you should also post diff's instead of full ebuilds

diff -u old-ebuild-1.0.ebuild new-ebuild-1.0-r1.ebuild > ebuild-diff.patch

[Comment 14 Simon Kohlmeyer 2012-09-03 19:04:09 UTC]

Sorry about the diffs.

Could you please explain why girara is slotted?

[Comment 15 Samuli Suominen (RETIRED) 2012-09-03 19:32:54 UTC]

girara can be built with either gtk+-2 or gtk+-3, and i'll add new SLOT="3" later on when it becomes useful

as in, all of girara and zathura can be built using gtk+-3 instead of gtk+-2 but it's still a bad idea because zathura-pdf-poppler plug-in only works with gtk+-2

if we had zathura-mupdf in portage, we could have gtk+-3 version of girara, zathura and zathura-mupdf

hope that clears it up

[Comment 16 Simon Kohlmeyer 2012-09-03 19:58:14 UTC]

Makes sense. I removed the slot because I thought it was not needed any longer.