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

Use a temporary directory for temporary object files #3643

Closed
kinke opened this issue Jan 15, 2021 · 0 comments
Closed

Use a temporary directory for temporary object files #3643

kinke opened this issue Jan 15, 2021 · 0 comments

Comments

@kinke
Copy link
Member

kinke commented Jan 15, 2021

With -cleanup-obj (e.g., implicitly injected by ldmd2 -lib), LDC should default to placing the temporary object files into a temporary directory (i.e., default -od value for object files only), not into the current working dir. Otherwise parallel compiler invocations in the same working dir can lead to obscure object file collisions, see atilaneves/reggae#117; DMD doesn't have this problem, and ldmd2 shouldn't either.
kinke added a commit to kinke/ldc that referenced this issue Feb 9, 2021
@kinke
-cleanup-obj: Put object files into temporary directory by default
79d4fd6 
To prevent conflicts for concurrent LDC processes in the same
working dir, resolving ldc-developers#3643.

Re-use this for the temporary object files of -run by enforcing
-cleanup-obj in that case. Additionally enforcing -oq prevents
conflicting filenames in that dir and allows to get rid of previous
Module.makeObjectFilenameUnique().
kinke added a commit to kinke/ldc that referenced this issue Feb 9, 2021
@kinke
-cleanup-obj: Put object files into temporary directory by default
8efdc33 
To prevent conflicts for concurrent LDC processes in the same
working dir, resolving ldc-developers#3643.

Re-use this for the temporary object files of -run by enforcing
-cleanup-obj in that case. Additionally enforcing -oq prevents
conflicting filenames in that dir and allows to get rid of previous
Module.makeObjectFilenameUnique().
@kinke kinke mentioned this issue Feb 9, 2021
Merged
kinke added a commit to kinke/ldc that referenced this issue Feb 10, 2021
@kinke
-cleanup-obj: Put object files into temporary directory by default
215c87c 
To prevent conflicts for concurrent LDC processes in the same
working dir, resolving ldc-developers#3643.

Re-use this for the temporary object files of -run by enforcing
-cleanup-obj in that case. Additionally enforcing -oq prevents
conflicting filenames in that dir and allows to get rid of previous
Module.makeObjectFilenameUnique().
kinke added a commit that referenced this issue Feb 10, 2021
@kinke
-cleanup-obj: Put object files into temporary directory by default (#…
8d71c86 
…3660)

To prevent conflicts for concurrent LDC processes in the same
working dir, resolving #3643.

Re-use this for the temporary object files of -run by enforcing
-cleanup-obj in that case. Additionally enforcing -oq prevents
conflicting filenames in that dir and allows to get rid of previous
Module.makeObjectFilenameUnique().
@kinke kinke closed this as completed Feb 11, 2021
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
1 participant
@kinke