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

app-i18n/ibus-libpinyin: Version bump 1.10.0 & fix boost configure #8986

Closed
wants to merge 10,000 commits into from

Conversation

gramsey
Copy link
Contributor

@gramsey gramsey commented Jun 27, 2018

1.9.3-r1 - This patches issue with latest GCC on configure with boost
1.10.0 - This includes above patch, and also update to new major version

I submitted the patch/fix upstream as pull request
libpinyin/ibus-libpinyin#134
and it has been merged (basically updating the boost.m4 macro to latest)

1.10.0 has additional changes as upstream has moved configuration settings
to gsettings (now requires gnome2_utils eclass)

Closes: https://bugs.gentoo.org/659328
Package-Manager: Portage-2.3.40, Repoman-2.3.9

floppym and others added 30 commits June 26, 2018 10:45
Package-Manager: Portage-2.3.40_p15, Repoman-2.3.9_p247
Package-Manager: Portage-2.3.40_p15, Repoman-2.3.9_p247
Package-Manager: Portage-2.3.40_p15, Repoman-2.3.9_p247
Closes: https://bugs.gentoo.org/659242
Package-Manager: Portage-2.3.40_p15, Repoman-2.3.9_p247
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Signed-off-by: Jan Vesely <jano.vesely@gmail.com>
Closes: gentoo#8966
Signed-off-by: Jan Vesely <jano.vesely@gmail.com>
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
klausman and others added 22 commits June 27, 2018 09:25
Package-Manager: Portage-2.3.41, Repoman-2.3.9
Package-Manager: Portage-2.3.41, Repoman-2.3.9
Package-Manager: Portage-2.3.41, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
1.9.3-r1 - This patches issue with latest GCC on configure with boost
1.10.0 - This includes above patch, and also update to new major version

I submitted the patch/fix upstream as pull request
libpinyin/ibus-libpinyin#134
and it has been merged (basically updating the boost.m4 macro to latest)

1.10.0 has additional changes as upstream has moved configuration settings
to gsettings (now requires gnome2_utils eclass)

Closes: https://bugs.gentoo.org/659328
Package-Manager: Portage-2.3.40, Repoman-2.3.9
@gentoo-bot
Copy link

Pull Request assignment

Areas affected: ebuilds
Packages affected: app-i18n/ibus-libpinyin

app-i18n/ibus-libpinyin: graham.ramsey[at]gmail.com, @gentoo/cjk, @gentoo/proxy-maint

Bugs linked: 659328

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Jun 27, 2018
}

pkg_preinst() {
gnome2_schemas_savelist
Copy link
Contributor

Choose a reason for hiding this comment

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

This is unnecessary. Please drop whole pkg_preinst().
Documentation of gnome2_schemas_savelist() in gnome2-utils.eclass says:

This is only necessary for eclass implementations that call gnome2_schemas_update conditionally.

PYTHON_COMPAT=( python{2_7,3_4,3_5,3_6} )
PYTHON_REQ_USE="sqlite(+)"

inherit autotools python-single-r1 gnome2-utils
Copy link
Contributor

Choose a reason for hiding this comment

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

inherit autotools gnome2-utils python-single-r1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks, makes sense - changes applied just as suggested.

sorted imports in alphabetical order
removed uneeded pkg_preinst / gnome2_schemas_savelist

Package-Manager: Portage-2.3.40, Repoman-2.3.9
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2018-06-28 07:54 UTC
Newest commit scanned: 376931f
Status: ✅ good

No issues found

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). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet