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

Check path validity on Windows will validate NTFS permissions for network drives #6209

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

allexzander
Copy link
Contributor

Based on #5510

Signed-off-by: Alkl58 [email protected]

@allexzander
Copy link
Contributor Author

@Alkl58 I had to create a new PR

Copy link

codecov bot commented Nov 8, 2023

Codecov Report

Merging #6209 (e30decc) into master (dd178f0) will decrease coverage by 0.04%.
Report is 4 commits behind head on master.
The diff coverage is 0.00%.

❗ Current head e30decc differs from pull request most recent head 9c68399. Consider uploading reports for the commit 9c68399 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6209      +/-   ##
==========================================
- Coverage   60.77%   60.74%   -0.04%     
==========================================
  Files         145      145              
  Lines       18836    18841       +5     
==========================================
- Hits        11448    11445       -3     
- Misses       7388     7396       +8     
Files Coverage Δ
src/common/utility.h 0.00% <ø> (ø)
src/common/utility_win.cpp 45.57% <0.00%> (-1.04%) ⬇️

... and 1 file with indirect coverage changes

@allexzander allexzander force-pushed the bugfix/test-check-ntfs-permission branch 3 times, most recently from a52fd47 to bc3beb5 Compare November 8, 2023 18:47
src/gui/folderman.cpp Outdated Show resolved Hide resolved
src/common/utility_win.cpp Outdated Show resolved Hide resolved
@allexzander allexzander force-pushed the bugfix/test-check-ntfs-permission branch 2 times, most recently from c54b780 to c7d28e6 Compare November 9, 2023 18:18
@allexzander allexzander force-pushed the bugfix/test-check-ntfs-permission branch from c7d28e6 to e30decc Compare November 9, 2023 20:23
@mgallien mgallien force-pushed the bugfix/test-check-ntfs-permission branch from e30decc to aa1b73f Compare November 10, 2023 15:04
@allexzander allexzander force-pushed the bugfix/test-check-ntfs-permission branch from aa1b73f to 9c68399 Compare November 13, 2023 11:00
@nextcloud-desktop-bot
Copy link

AppImage file: nextcloud-PR-6209-9c6839969cd77b8fbb5064a5913a22f6d66d0fbe-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

Copy link

sonarcloud bot commented Nov 13, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@Alkl58
Copy link
Contributor

Alkl58 commented Nov 13, 2023

Builds are failing because of the following reason:

[Errno 2] No such file or directory: 'meson'
Traceback (most recent call last):
  File "/Users/runner/work/desktop/desktop/macos-64-clang/craft/bin/craft.py", line 417, in <module>
    success = main(timer)
  File "/Users/runner/work/desktop/desktop/macos-64-clang/craft/bin/craft.py", line 399, in main
    if not CraftCommands.run(package, action, tempArgs):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/CraftCommands.py", line 458, in run
    if not handlePackage(info, action, directTargets=directTargets):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/CraftCommands.py", line 111, in handlePackage
    if not doExec(package, action):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/CraftCommands.py", line 60, in doExec
    ret = package.instance.runAction(action)
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/Package/PackageBase.py", line 275, in runAction
    if not getattr(self, step)():
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/BuildSystem/MesonBuildSystem.py", line 127, in configure
    if not utils.system(Arguments(["meson", "setup", extra_options, self.configureOptions(defines)])):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/utils.py", line 218, in system
    return systemWithoutShell(
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/utils.py", line 325, in systemWithoutShell
    proc = subprocess.Popen(cmd, **kw)
  File "/Users/runner/hostedtoolcache/Python/3.9.18/x64/lib/python3.9/subprocess.py", line 951, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/Users/runner/hostedtoolcache/Python/3.9.18/x64/lib/python3.9/subprocess.py", line 1837, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'meson'

Which is an issue during the dependency install stage - the worker doesn't even reach the building state.

@allexzander allexzander merged commit e2f6b5f into master Nov 13, 2023
10 of 12 checks passed
@allexzander allexzander deleted the bugfix/test-check-ntfs-permission branch November 13, 2023 13:18
@allexzander
Copy link
Contributor Author

Builds are failing because of the following reason:

[Errno 2] No such file or directory: 'meson'
Traceback (most recent call last):
  File "/Users/runner/work/desktop/desktop/macos-64-clang/craft/bin/craft.py", line 417, in <module>
    success = main(timer)
  File "/Users/runner/work/desktop/desktop/macos-64-clang/craft/bin/craft.py", line 399, in main
    if not CraftCommands.run(package, action, tempArgs):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/CraftCommands.py", line 458, in run
    if not handlePackage(info, action, directTargets=directTargets):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/CraftCommands.py", line 111, in handlePackage
    if not doExec(package, action):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/CraftCommands.py", line 60, in doExec
    ret = package.instance.runAction(action)
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/Package/PackageBase.py", line 275, in runAction
    if not getattr(self, step)():
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/BuildSystem/MesonBuildSystem.py", line 127, in configure
    if not utils.system(Arguments(["meson", "setup", extra_options, self.configureOptions(defines)])):
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/utils.py", line 218, in system
    return systemWithoutShell(
  File "/Users/runner/work/desktop/desktop/craft-clone/bin/utils.py", line 325, in systemWithoutShell
    proc = subprocess.Popen(cmd, **kw)
  File "/Users/runner/hostedtoolcache/Python/3.9.18/x64/lib/python3.9/subprocess.py", line 951, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/Users/runner/hostedtoolcache/Python/3.9.18/x64/lib/python3.9/subprocess.py", line 1837, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'meson'

Which is an issue during the dependency install stage - the worker doesn't even reach the building state.

thanks, we will need to check the Craft part

@allexzander
Copy link
Contributor Author

not releasing, as it brings issues with VFS placeholders

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.

5 participants