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

Can't have as a mandatory dependency a deprecated system #108

Closed
wants to merge 1 commit into from

Conversation

aleixpol
Copy link
Collaborator

The sources aren't available by the author

fedorahosted is deprecated and there's no alternative offered by upstream.

The sources aren't available by the author
@ximion
Copy link
Owner

ximion commented Mar 20, 2017

Just making this a warning will not make the tool not-use xmlto.
I'll need to look into what it does and how to maybe replace it (the number of tools depending on it is quite huge though, so it won't go away anytime soon).
I do wonder about the weird way this tested if the variable was set before... Not sure why this was done the way it is.

@ximion
Copy link
Owner

ximion commented Apr 11, 2017

I'll reject the PR for now, given that xmlto is under active (but very slow) development at https://pagure.io/xmlto and widely used in pretty much all distributions, and the PR will just break manpage generation on systems which don't have xmlto.

Long-term, we could think of a different way to make manpages from Docbook XML though.

@ximion ximion closed this Apr 11, 2017
ximion added a commit that referenced this pull request Apr 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants