Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 368491 - New package: x11-terms/valaterm
Summary: New package: x11-terms/valaterm
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: Normal enhancement (vote)
Assignee: Samuli Suominen (RETIRED)
URL: http://git-jpdeplaix.dyndns.org/valat...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-23 19:34 UTC by Jacques-Pascal Deplaix
Modified: 2012-06-24 19:29 UTC (History)
0 users

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


Attachments
valaterm-0.3.ebuild (valaterm-0.3.ebuild,796 bytes, text/plain)
2011-05-23 19:36 UTC, Jacques-Pascal Deplaix
Details
valaterm-0.3-r1.ebuild (valaterm-0.3-r1.ebuild,816 bytes, text/plain)
2011-05-24 12:51 UTC, Jacques-Pascal Deplaix
Details
valaterm-0.4.2.ebuild (valaterm-0.4.2.ebuild,931 bytes, text/plain)
2011-09-18 20:02 UTC, Jacques-Pascal Deplaix
Details
valaterm-0.5.ebuild (valaterm-0.5.ebuild,940 bytes, text/plain)
2012-06-21 10:24 UTC, Jacques-Pascal Deplaix
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jacques-Pascal Deplaix 2011-05-23 19:34:59 UTC
ValaTerm is a lightweight terminal written in Vala.

Reproducible: Always
Comment 1 Jacques-Pascal Deplaix 2011-05-23 19:36:30 UTC
Created attachment 274429 [details]
valaterm-0.3.ebuild
Comment 2 Jacques-Pascal Deplaix 2011-05-24 12:51:22 UTC
Created attachment 274473 [details]
valaterm-0.3-r1.ebuild
Comment 3 Jacques-Pascal Deplaix 2011-09-18 20:02:30 UTC
Created attachment 286915 [details]
valaterm-0.4.2.ebuild
Comment 4 Samuli Suominen (RETIRED) gentoo-dev 2012-01-14 16:50:51 UTC
0.4.3 is in portage now
Comment 5 Jacques-Pascal Deplaix 2012-06-21 10:24:27 UTC
Created attachment 315913 [details]
valaterm-0.5.ebuild
Comment 6 Samuli Suominen (RETIRED) gentoo-dev 2012-06-22 08:37:51 UTC
This bug was for initial introduction of valaterm to Portage which has been completed. Do not reopen bugs like this. Version bumps go to new bug.
Comment 7 Jacques-Pascal Deplaix 2012-06-22 09:31:54 UTC
Ok, I'm sorry for that.
Can I have some explanations for your modifications on my ebuild ?
1) LICENSE="GPL-2" is wrong. ValaTerm is under GPL-3

2) src_prepare() is no longer necessary because of this commit: http://git-jpdeplaix.dyndns.org/valaterm/valaterm.git/commit/?id=4a8ecb533d3cd35650f1918442cfff035ec3a88b

3) Why did you remove the gtk3 USE flag ?
Comment 8 Samuli Suominen (RETIRED) gentoo-dev 2012-06-22 10:08:48 UTC
(In reply to comment #7)
> Ok, I'm sorry for that.

np

> Can I have some explanations for your modifications on my ebuild ?
> 1) LICENSE="GPL-2" is wrong. ValaTerm is under GPL-3

Fixed. I think I've missed a change in the COPYING file or misread it.
> 
> 2) src_prepare() is no longer necessary because of this commit:
> http://git-jpdeplaix.dyndns.org/valaterm/valaterm.git/commit/
> ?id=4a8ecb533d3cd35650f1918442cfff035ec3a88b

That will still prepend -s to LINKFLAGS which causes prestripping of the binary. Stripping is handled by Portage via FEATURES.

Then it prepends -O optimization flag if user has no CFLAGS defined on his own.

If it was only the CFLAGS case, I'd propably let it slip through, and no src_prepare() would be required, but figured while removing the -s from LINKFLAGS, I might add couple of more expressions in the same `sed` call "while at it"

> 
> 3) Why did you remove the gtk3 USE flag ?

Because valaterm is not a library with reverse dependencies which would still require different GTK+ version
Even then it would be preferred to have /usr/lib64/libvalaterm-gtk2.so and libvalaterm-gtk3.so and they would be in SLOT="2" of valaterm-0.5-r200 and SLOT="3" of valaterm-0.5-r300
As in, adding versioned gtk USE flag is the absolute last resort and in fact has no valid use cases in Portage at all, and the ones that have such USE flag are bugs
See bug 374057 and bug 420493

So conclusion... We can add 2 SLOT's of valaterm in Portage if valaterm with GTK+-2 has some features that GTK+-3 version doesn't and that they are important enough to cost the extra maintaince burden
Comment 9 Jacques-Pascal Deplaix 2012-06-24 19:29:24 UTC
(In reply to comment #8)
> That will still prepend -s to LINKFLAGS which causes prestripping of the
> binary. Stripping is handled by Portage via FEATURES.
> 
> Then it prepends -O optimization flag if user has no CFLAGS defined on his
> own.
> 
> If it was only the CFLAGS case, I'd propably let it slip through, and no
> src_prepare() would be required, but figured while removing the -s from
> LINKFLAGS, I might add couple of more expressions in the same `sed` call
> "while at it"
>

Ok, I changed that in this commit: http://git-jpdeplaix.dyndns.org/valaterm/valaterm.git/commit/?id=a7530301784601ab0da1a41dd088e1b16a5c7fdb
It will be available in the next release.