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

Legacy / Settings Relative Paths #981

Merged

Conversation

jcolicchio
Copy link
Contributor

@jcolicchio jcolicchio commented Nov 2, 2020

Fixes #979

This PR causes included yaml to adjust LegacyTarget's workingDirectory to be relative to the including yaml file

@jcolicchio jcolicchio marked this pull request as draft November 2, 2020 01:35
@jcolicchio
Copy link
Contributor Author

@yonaskolb I'd actually appreciate your input here on this "fix" before I add tests. It's something that'd benefit my use case, but I'm not sure if it's generally a good idea when including other yaml files, that we change the HEADER_SEARCH_PATHS to be relative to the new location. Thoughts?

@yonaskolb
Copy link
Owner

Hi @jcolicchio, thanks for this. Yeah I'm a bit hesitant to apply changes to HEADER_SEARCH_PATHS, especially as it would be a breaking change for many. I'm happy to include the working directory though as that's an oversight, even though it's also technically breaking, which the changelog entry can indicate with Breaking

@jcolicchio jcolicchio marked this pull request as ready for review January 11, 2021 17:29
@jcolicchio
Copy link
Contributor Author

@yonaskolb sounds good! I've undone my change to HEADER_SEARCH_PATHS and updated the changelog 👍

69078D1DA3F942D5BC752081 /* Sources */,
);
buildToolPath = /usr/bin/true;
buildWorkingDirectory = .;
Copy link
Owner

@yonaskolb yonaskolb Jan 12, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this supposed to be AnotherProject, or is this correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is correct — when generating AnotherProject directly from the AnotherProject folder, the working directory for the target is ‘.’, but when including it in another project in a different directory, that project’s target will have a working directory relative to the other project.

In other words, regardless of where the generated project ends up, the working directory for the target declared in this folder will evaluate to this folder

@yonaskolb yonaskolb merged commit 0eeb636 into yonaskolb:master Jan 14, 2021
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.

Relative Paths for Included Settings, Legacy workingDirectory
2 participants