Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x11-libs/vte version bump to 0.28.2-r208 #2444

Closed
wants to merge 1 commit into from
Closed

x11-libs/vte version bump to 0.28.2-r208 #2444

wants to merge 1 commit into from

Conversation

djarosz
Copy link
Contributor

@djarosz djarosz commented Oct 1, 2016

@gentoo/gnome

Changes

- version bump
- included patch for vte2 fixing bug described below
- added call to epatch_user

Bug details

If you are using gnome-terminal or xfce4-terminal, you may have hit a bug in vte3.
A fix is around since at least 2010, which found its way into some
(but not all) distribution packages, and even into upstream (3 years later,
but unchanged).

Bugzilla and other links

@djarosz
Copy link
Contributor Author

djarosz commented Oct 1, 2016

Gentto bug report: https://bugs.gentoo.org/show_bug.cgi?id=595770
Sorry cannot assign Gnome team thou.

@doughdemon
Copy link
Contributor

  • The stable keywords should be removed for a revbump.
  • There does not seem any need for this package to use the python-r1 eclass. The ebuild would become much simpler with python-single-r1.

@monsieurp monsieurp added version bump assigned PR successfully assigned to the package maintainer(s). labels Oct 1, 2016
@doughdemon
Copy link
Contributor

Sorry that I was not clear. With converting to python_single-r1, much of the ebuild can be simplified. I think that it can basically be made to look like the ebuild for version 0.36.5, except for an additional $(use_enable python) in src_prepare(). (If you are not comfortable with the conversion, please better keep the python-r1 eclass.)

@djarosz
Copy link
Contributor Author

djarosz commented Oct 4, 2016

@doughdemon sorry I did quite understand what you ment before. My primary intention was to include patch for reported bug. I do not feel comfortable in changing anything else in this ebuild since I'm not an export in this area and changing without good understanding can have impact on package stability.

So I will just revet python-single-r1 to python-r1

Copy link
Member

@mgorny mgorny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What @doughdemon -- please switch it back to python-r1, or update for python-single-r1 completely. You're calling the functions from the wrong eclass right now, i.e. it can't work. Also, please squash this into a single commit, there is no reason to split it here.

	- version bump
	- included patch for vte2 fixing bug described below
	- added call to epatch_user

If you are using gnome-terminal or xfce4-terminal, you may have hit a bug in vte3.
A fix is around since at least 2010, which found its way into some
(but not all) distribution packages, and even into upstream (3 years later,
but unchanged).

Upstream bugzilla and other links
- Original (?):
	https://bugzilla.gnome.org/show_bug.cgi?id=542087
- One of its duplicate:
	https://bugzilla.gnome.org/show_bug.cgi?id=686097
- reported mosh bug related to this problem:
	mobile-shell/mosh#660
- vte upstream commit:
	https://git.gnome.org/browse/vte/commit/?id=88e8e89560a62d0981ce2b18974a230d0a07dbdd
- used backported patch for vte2:
	pld-linux/vte0@1e8dce1
@djarosz
Copy link
Contributor Author

djarosz commented Dec 7, 2016

@mgorny reverted to python-r1 and squashed

@pacho2
Copy link
Member

pacho2 commented Dec 7, 2016

You can also drop the DOCS specific statement with eapi6 and gnome2.eclass usage

Additionally, if I don't misremember you need to also provide in the PR the updated Manifest for allowing us to directly push it... but maybe I am wrong as I am still not fully used to github workflow, sorry if that is the case

Copy link
Member

@mgorny mgorny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see anything really wrong. Minor nit wrt empty line and the commit message needs to be adjusted for our standards (it's not a version bump, meaningful summary, tag-like URLs). Both can be done by the person merging it.

DOCS="AUTHORS ChangeLog HACKING NEWS README"

src_prepare() {

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This empty line needs to be removed.

@mgorny
Copy link
Member

mgorny commented Dec 8, 2016

@gentoo/gnome, any more comments here? Should I fix the commit messages and the empty line and merge, or does one of you want to?

@pacho2
Copy link
Member

pacho2 commented Dec 9, 2016

If you have time to do, please go ahead

And thanks a lot!

@gentoo-bot gentoo-bot closed this in 3396864 Dec 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s).
Projects
None yet
9 participants