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

net-misc/icaclient: bump to 22.12.0.12 #28493

Closed

Conversation

henning-schild
Copy link
Contributor

Here we do not just bump the version but also change the way we deal with the shared config file between this ebuild and its plugin media-plugins/hdx-realtime-media-engine.
Also we move the rdep to sys-libs/llvm-libunwind to the plugin. And if the user wants hdx support on icaclient they have to set a USE flag. The plugin will not work if icaclient is not built with support for it and the plugin itself contains the tool to enable itself so we turn the dep chain around, the plugin is even a bdep of the application.

Closes: https://bugs.gentoo.org/883775
Closes: https://bugs.gentoo.org/881637
Signed-off-by: Henning Schild henning@hennsch.de

Here we do not just bump the version but also change the way we deal
with the shared config file between this ebuild and its plugin
media-plugins/hdx-realtime-media-engine.
Also we move the rdep to sys-libs/llvm-libunwind to the plugin. And if
the user wants hdx support on icaclient they have to set a USE flag. The
plugin will not work if icaclient is not built with support for it and
the plugin itself contains the tool to enable itself so we turn the dep
chain around, the plugin is even a bdep of the application.

Closes: https://bugs.gentoo.org/883775
Closes: https://bugs.gentoo.org/881637
Signed-off-by: Henning Schild <henning@hennsch.de>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @henning-schild
Areas affected: ebuilds
Packages affected: media-plugins/hdx-realtime-media-engine, net-misc/icaclient

media-plugins/hdx-realtime-media-engine: @henning-schild, @gentoo/proxy-maint
net-misc/icaclient: @henning-schild, @gentoo/proxy-maint

Linked bugs

Bugs linked: 881637, 883775


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

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Dec 1, 2022
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-12-01 15:33 UTC
Newest commit scanned: 195d199
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/8e411a6504/output.html

@henning-schild
Copy link
Contributor Author

@juippis this is open for a while now, not a simple bump but i would like to ask you to review, you have been involved in other PRs on that package

xdg_mimeinfo_database_update

if ! use hdx; then
if [ -x "${ROOT}${ICAROOT}"/rtme/RTMEconfig ]; then
Copy link
Member

Choose a reason for hiding this comment

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

Use bash tests ([[ ]])

default
rm lib/UIDialogLibWebKit.so || die

cp nls/en/module.ini .
Copy link
Member

Choose a reason for hiding this comment

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

missing die on cp, rtmeconfig call, mv and mv.

@juippis
Copy link
Member

juippis commented Jan 7, 2023

Sorry I was away with a travel laptop that isn't suited for building these PR packages, and put my full focus on going through my unread bugzilla mail. Glad to see @thesamesam was able to process this!

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. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
5 participants