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

file permission transfer from source template won't work on windows #141

Closed
chfw opened this issue Dec 10, 2018 · 1 comment · Fixed by #161
Closed

file permission transfer from source template won't work on windows #141

chfw opened this issue Dec 10, 2018 · 1 comment · Fixed by #161

Comments

@chfw
Copy link
Member

chfw commented Dec 10, 2018

2018-12-10T22:33:56.0900746Z ======================================================================
2018-12-10T22:33:56.0900924Z FAIL: Save as above, but this time,
2018-12-10T22:33:56.0901119Z ----------------------------------------------------------------------
2018-12-10T22:33:56.0901293Z Traceback (most recent call last):
2018-12-10T22:33:56.0901495Z   File "c:\hostedtoolcache\windows\python\3.7.0\x64\lib\site-packages\nose\case.py", line 198, in runTest
2018-12-10T22:33:56.0901676Z     self.test(*self.arg)
2018-12-10T22:33:56.0901864Z   File "D:\a\1\s\tests\test_hash_store.py", line 94, in test_dest_file_file_permision_changed
2018-12-10T22:33:56.0902217Z     assert flag is True
2018-12-10T22:33:56.0902393Z AssertionError

https://dev.azure.com/moremoban/moban/_build/results?buildId=16

chfw added a commit that referenced this issue Dec 10, 2018
chfw added a commit that referenced this issue Dec 10, 2018
…:wheel_chair: skip the file permission related tests for now. #141
chfw added a commit that referenced this issue Dec 10, 2018
…:wheel_chair: skip the file permission related tests for now. #141
@chfw chfw mentioned this issue Dec 10, 2018
@chfw
Copy link
Member Author

chfw commented Dec 16, 2018

@jayvdb , what your opinion on permission transfer on windows os?

I would say no to transfer unix file permission to windows file when apply unix based repo on windows files.

chfw added a commit that referenced this issue Jan 11, 2019
This was referenced Jan 11, 2019
@chfw chfw closed this as completed in #161 Jan 12, 2019
ayan-b pushed a commit to ayan-b/moban that referenced this issue Jan 15, 2019
…ban#122. :wheel_chair: skip the file permission related tests for now. moremoban#141
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 a pull request may close this issue.

1 participant