Skip to content
This repository has been archived by the owner on Jan 31, 2022. It is now read-only.

setup scripts do naughty things #109

Closed
jsturdy opened this issue Aug 9, 2017 · 0 comments
Closed

setup scripts do naughty things #109

jsturdy opened this issue Aug 9, 2017 · 0 comments
Assignees
@bdorney @lmoureaux
Labels
Priority: Medium Type: Bug

Comments

@jsturdy
Copy link
Contributor

jsturdy commented Aug 9, 2017

Sourcing the setup/paths.sh script leaves the user in a specific directory, which is not good practice, especially when used in a profile init script
Probably everything below #L44 should not do a cd
@lmoureaux lmoureaux mentioned this issue Aug 9, 2017
Merged
@bdorney bdorney mentioned this issue Aug 10, 2017
Closed
@jsturdy jsturdy closed this as completed Aug 11, 2017
AndrewLevin pushed a commit to AndrewLevin/vfatqc-python-scripts that referenced this issue May 19, 2020
@lmoureaux
Drop references to vfatqc from the README
de53e70 
See PR cms-gem-daq-project#109
AndrewLevin pushed a commit to AndrewLevin/vfatqc-python-scripts that referenced this issue May 19, 2020
@lmoureaux
Tell the user where the tags can be found
d269180 
And how their syntax maps to the -c/-g/-G options.

Change requested by Brian on PR cms-gem-daq-project#109
AndrewLevin pushed a commit to AndrewLevin/vfatqc-python-scripts that referenced this issue May 19, 2020
@lmoureaux
Check that gemPlotter output files exist
67ccbf8 
See PR cms-gem-daq-project#109
AndrewLevin pushed a commit to AndrewLevin/vfatqc-python-scripts that referenced this issue May 19, 2020
@bdorney
Merge pull request cms-gem-daq-project#109 from lmoureaux/fix_README
29fb355 
Resolves cms-gem-daq-project#96
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bdorney bdorney

@lmoureaux lmoureaux

Labels
Priority: Medium Type: Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jsturdy @bdorney @lmoureaux