Skip to content

Commit

Permalink
try out potential fix for missing "everyone" ACE in fs.smbfs
Browse files Browse the repository at this point in the history
  • Loading branch information
telamonian committed Apr 14, 2020
1 parent 5e796e1 commit f3079d5
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ install:

test_script:
# uncomment to allow remote login to windows ci vm via rdp
- ps: $blockRdp = $true; iex ((new-object net.webclient).DownloadString('https://raw.githubusercontent.com/appveyor/ci/master/scripts/enable-rdp.ps1'))
# - ps: $blockRdp = $true; iex ((new-object net.webclient).DownloadString('https://raw.githubusercontent.com/appveyor/ci/master/scripts/enable-rdp.ps1'))

# lint the python code
# - python -m flake8 jupyterfs setup.py
Expand Down
4 changes: 2 additions & 2 deletions jupyterfs/tests/test_pyfilesystem_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,9 @@
test_hostname_smb_docker_share = 'TESTNET'
test_name_port_smb_docker_share = 3669

test_direct_tcp_smb_os_share = True
test_direct_tcp_smb_os_share = False
test_host_smb_os_share = socket.gethostbyname_ex(socket.gethostname())[2][-1]
test_smb_port_smb_os_share = 445
test_smb_port_smb_os_share = 139

_test_file_model = {
'content': test_content,
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
'boto3',
'docker',
'fs-miniofs',
'fs.smbfs @ git+https://github.com/telamonian/fs.smbfs.git@host-name-arg#egg=fs.smbfs',
'fs.smbfs @ git+https://github.com/telamonian/fs.smbfs.git@dont_assume_everyone_ace_exists#egg=fs.smbfs',
'mock',
'pysmb',
'pytest',
Expand Down

0 comments on commit f3079d5

Please sign in to comment.