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.

Sign up for GitHub

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

Jump to bottom

Correct man path #62

Closed
ford--prefect opened this issue Mar 14, 2022 · 4 comments · Fixed by #72
Closed

Correct man path #62

ford--prefect opened this issue Mar 14, 2022 · 4 comments · Fixed by #72

Comments

@ford--prefect
Copy link

Currently, in setup.py#L17 the man path is incorrect.
Instead of man/man1 it should be share/man/man1.
@cosmosgenius
Copy link
Member

Could you provide some links to validate the change. The current implementation is based on nose repo https://github.com/nose-devs/nose/blob/master/setup.py#L109

@ford--prefect
Copy link
Author

The man-page of the file system hierarchy might serve as sufficiently credible source?

@ford--prefect ford--prefect mentioned this issue Mar 14, 2022
Open
@chenrui333
Copy link
Contributor

yeah, the man path is incorrect.

@chenrui333 chenrui333 mentioned this issue Nov 27, 2023
Merged
@subho007
Copy link
Member

subho007 commented Dec 5, 2023

This is released: https://pypi.org/project/pyaxmlparser/0.3.30/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: use std man page chenrui333/pyaxmlparser
4 participants
@subho007 @cosmosgenius @chenrui333 @ford--prefect