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

remove the symbolic link in the meson test #697

Merged
merged 5 commits into from
Jan 28, 2022

Conversation

phlptp
Copy link
Collaborator

@phlptp phlptp commented Jan 27, 2022

Remove the symbolic link for Issue #696

Add a symlink generation command in the meson CI test

@phlptp phlptp force-pushed the remove_symbolic_link branch 4 times, most recently from d4277ae to 0523377 Compare January 27, 2022 21:43
@phlptp phlptp force-pushed the remove_symbolic_link branch 2 times, most recently from f208055 to d1f9b90 Compare January 27, 2022 22:46
@henryiii
Copy link
Collaborator

Calling it .gitkeep is somewhat standard. And you weren't able to mkdir it, I guess?

@phlptp
Copy link
Collaborator Author

phlptp commented Jan 27, 2022

trying the mkdir now

@phlptp
Copy link
Collaborator Author

phlptp commented Jan 27, 2022

looks like the mkdir from the command line works as well, so that is probably the best solution here

@phlptp phlptp requested a review from henryiii January 27, 2022 23:28
@jspelletier
Copy link

jspelletier commented Jan 28, 2022

Note: the symlink should be deleted after the test or the problem with gitlab will persist if someone has a gitlab CI executing that test as a submodule. in my case since I am not executing the tests, I think that would work

@henryiii
Copy link
Collaborator

gitlab CI executing that test as a submodule

Why would gitlab CI pick up a line from Azure pipelines?

azure-pipelines.yml Outdated Show resolved Hide resolved
Co-authored-by: Henry Schreiner <HenrySchreinerIII@gmail.com>
@phlptp phlptp merged commit ba5ca8c into CLIUtils:main Jan 28, 2022
@phlptp phlptp deleted the remove_symbolic_link branch January 28, 2022 14:49
@github-actions github-actions bot added needs changelog Hasn't been added to the changelog yet needs README Needs to be mentioned in the README labels Jan 28, 2022
@henryiii henryiii removed needs changelog Hasn't been added to the changelog yet needs README Needs to be mentioned in the README labels Feb 9, 2022
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.

3 participants