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.

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

Use file in path #138

Closed
wants to merge 2 commits into from
Closed

Use file in path #138

wants to merge 2 commits into from

Conversation

pfs
Copy link

@pfs pfs commented Aug 21, 2024

Resolve full path using file in path, otherwise jobs will file (e.g. running from any directory other then $CMSSW_BASE/src as in a HTCondor node)

@IzaakWN
Copy link

IzaakWN commented Aug 21, 2024

This looks fine to me.

This was actually one of Matti's comments. Do you need this commit before next week? Otherwise you could change the target branch to the one for PR #137, i.e. branch dev/CMSSW_14_1_X_PR_suggestions, so we can group together all code suggestions.

@IzaakWN
Copy link

IzaakWN commented Aug 27, 2024

I added this to #137.

@pfs
Copy link
Author

pfs commented Aug 27, 2024

Deprecated

@pfs pfs closed this Aug 27, 2024
@pfs pfs deleted the dev/usefip branch August 27, 2024 13:49
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.

2 participants