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
dev-libs/xqilla: version bump to 2.3.4 #11179
Conversation
ef929b7
to
62de0a6
Compare
Copyright policy changePlease note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article. Pull Request assignmentSubmitter: @jok-ts dev-libs/xqilla: @gentoo/proxy-maint (maintainer needed) Linked bugsIn order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also include taking over maintenance and removing the p.mask (in separate commits).
dev-libs/xqilla/metadata.xml
Outdated
@@ -7,6 +7,8 @@ | |||
certain tasks</flag> | |||
<flag name="htmltidy">Use <pkg>app-text/htmltidy</pkg> when parsing | |||
HTML</flag> | |||
<flag name="tidy">Use <pkg>app-text/tidy-html5</pkg> when parsing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a global flag, so you don't need to describe it again.
dev-libs/xqilla/xqilla-2.3.4.ebuild
Outdated
} | ||
|
||
src_install () { | ||
use doc && HTML_DOCS=( docs/{dev-api,dom3-api,simple-api} ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can make the var local
.
dev-libs/xqilla/xqilla-2.3.4.ebuild
Outdated
use doc && HTML_DOCS=( docs/{dev-api,dom3-api,simple-api} ) | ||
default | ||
|
||
if ! use static-libs; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you really need .la
files for static libs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably not, I just tried to do this with minimal changes to the existing ebuild.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also include taking over maintenance and removing the p.mask (in separate commits).
I don't think I want to take over the maintenance.
Also, depend on app-text/tidy-html5 instead of app-text/htmltidy Signed-off-by: Jouni Kosonen <jouni.kosonen@tukesoft.com> Closes: https://bugs.gentoo.org/672886 Closes: https://bugs.gentoo.org/671442
Pull request CI reportReport generated at: 2019-04-21 00:56 UTC No issues found |
I'm afraid in this case I don't see the purpose of merging this as primary last rite reason (lack of maintainer) still stands. Merging it would be just patch work that would result in some other issue setting up back for removal later on. |
Well, got three out of four sorted. I'll close this PR later. |
Closing after package cleanup. |
Closes: https://bugs.gentoo.org/672886
dev-libs/xqilla-2.3.3 doesn't compile against dev-libs/xerces-c-3.2.2-r1 which just went stable everywhere.
Version bump to a version that does, only the subslot and the xerces-c dependency have been changed.