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

Looks like python/lvs_manager.py may have been unintentionally added to the repo. #44

Open
d-m-bailey opened this issue Dec 30, 2021 · 1 comment
Open

Looks like python/lvs_manager.py may have been unintentionally added to the repo. #44

d-m-bailey opened this issue Dec 30, 2021 · 1 comment

Comments

@d-m-bailey
Copy link
Contributor

In commit 80947400487c6447bd42574cf383e6a0cce2244c, python/lvs_manager.py may have been inadvertently added to the repo. This file is automatically created in the python/Makefile here

lvs_manager.py: lvs_manager.py.in
        sed -e '/SUBST_SCRIPT_DIR/s#SUBST_SCRIPT_DIR#$(SCRIPTINSTALL)#' \
                lvs_manager.py.in > lvs_manager.py

It should probably be added to .gitignore with similar files already there such as tcltk/netgen.tcl
@RTimothyEdwards
Copy link
Owner

Yup, thanks for pointing that out. I just fixed it.

RTimothyEdwards added a commit that referenced this issue Dec 31, 2021
@RTimothyEdwards
Removed lvs_manager.py, which is a derived file and should not have
e07a5b4 
ended up in the repository, as pointed out by Mitch Bailey in github
issue #44.  Added lvs_manager.py to .gitignore to prevent that from
happening again in the future.
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
@d-m-bailey @RTimothyEdwards