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

Add realpath for $mock_cache_path var, fix tests/code when PrivateTmp=on in systemd on mac enabled #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

proggga
Copy link
Contributor

@proggga proggga commented Nov 6, 2020

I got failed tests on my mac

So I made a little research and found a problem here.
Test compares real path in /private/tmp, and path that we concat.
They are differ (cause Mac had set PrivateTmp=on in systems), so I've added realdir to fix this behaviour

Also fixed path for lock file

image

@proggga proggga force-pushed the fix_systemd_PrivateTmp branch from 4d1b9da to 4ccb510 Compare November 6, 2020 23:21
@proggga
Copy link
Contributor Author

proggga commented Nov 7, 2020

I will rebase, after #51 will be merged. (Contains composer2 fix - special fix for travis)

@proggga proggga changed the title Fix path resolution for dir when PrivateTmp=on Add realpath for $mock_cache_path var, fix tests/code when PrivateTmp=on in systemd on mac enabled Nov 7, 2020
@proggga proggga force-pushed the fix_systemd_PrivateTmp branch from 4ccb510 to 4b216e8 Compare February 8, 2021 12:32
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