Skip to content

Conversation

@boffman
Copy link

@boffman boffman commented May 4, 2025

What does it implement/fix? Explain your changes.

I made a xsstrike package, and moved/refactored the other packages as sub-packages to it, and added a setup.py with an entrypoint, making it possible to install locally and just run xsstrike instead of python <path_to_xsstrike.py>.

That required updating some file reading paths for using the package instead of relative path to the script, plus assorted changes to imports. But now it can be installed directly from github with e.g. pip install --user git+https://github.com/..../repo.git and you get the script in $PATH which I think is nice.

xsstrike.py was renamed/moved to xsstrike/cli.py and a main() was added in it, which is referenced in setup.py as the entrypoint for the package.

Where has this been tested?

Python Version:\ 3.12
Operating System: Ubuntu Linux 24.04

Does this close any currently open issues?

No

Does this add any new dependency?

No

Does this add any new command line switch/option?

No

Any other comments you would like to make?

I guess this has impact on documentation/images to run xsstrike instead of python xsstrike.py .. I like it this way myself, having it in PATH, and offer this PR if it's interesting also in the main repo :-)

Some Questions

  • [ x] I have documented my code.
  • I have tested my build before submitting the pull request.

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.

1 participant