Skip to content

fix: quote external QM paths - #363

Closed
galjos wants to merge 1 commit into
fix/external-qm-failuresfrom
fix/external-qm-path-quoting
Closed

fix: quote external QM paths#363
galjos wants to merge 1 commit into
fix/external-qm-failuresfrom
fix/external-qm-path-quoting

Conversation

@galjos

@galjos galjos commented Jul 28, 2026

Copy link
Copy Markdown
Member

Closes #362.

Depends on #335.

  • add POSIX-safe shell argument quoting
  • quote DFTB+, PySCF, and Turbomole paths
  • cover spaces, apostrophes, and shell metacharacters

Tests:

  • testExternalQMRunner
  • testStringUtilities

@galjos

galjos commented Jul 28, 2026

Copy link
Copy Markdown
Member Author

Superseded by #384, which combines external-QM process and path handling.

@galjos galjos closed this Jul 28, 2026
@galjos
galjos deleted the fix/external-qm-path-quoting branch August 3, 2026 19: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.

1 participant