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

the ansi2html scripts collides with the ansi2html package #8

Closed
jlec opened this issue Jun 6, 2015 · 2 comments
Closed

the ansi2html scripts collides with the ansi2html package #8

jlec opened this issue Jun 6, 2015 · 2 comments

Comments

@jlec
Copy link

jlec commented Jun 6, 2015

What is the purpose of this package installing the script under ansi2html instead of coloredlogs or so?
minaco2 pushed a commit to git-portage/git-portage that referenced this issue Jun 7, 2015
drop ansi2html script due to filecollision (xolox/python-coloredlogs#8)…
cafe46a 
…, bug #551298

(Portage version: 2.2.20/cvs/Linux x86_64, signed Manifest commit with key E9402A79B03529A2!)
@xolox xolox changed the title the ansi2html scripts collids with the ansi2html package the ansi2html scripts collides with the ansi2html package Oct 6, 2015
@xolox xolox closed this as completed in eaef420 Oct 14, 2015
@xolox
Copy link
Owner

xolox commented Oct 14, 2015

Hi Justin and thanks for the feedback!

When I originally picked the ansi2html name I clearly didn't put much thought into name collisions, sorry about that. As you suggested I've now changed the name of the command line program to coloredlogs, with the ANSI to HTML conversion exposed as coloredlogs --to-html.

I hope/assume this resolves your issue and obviates the need to patch the setup.py script.

@jlec
Copy link
Author

jlec commented Oct 15, 2015

Thanks a bunch! Looks good. I will bump the version for Gentoo today.

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

No branches or pull requests
2 participants
@jlec @xolox