Skip to content

Allow creation of symlink to non-existent target - #804

Merged
Andy Jordan (andyleejordan) merged 1 commit into
masterfrom
andschwa/symbolic-target
Apr 10, 2016
Merged

Allow creation of symlink to non-existent target#804
Andy Jordan (andyleejordan) merged 1 commit into
masterfrom
andschwa/symbolic-target

Conversation

@andyleejordan

@andyleejordan Andy Jordan (andyleejordan) commented Apr 10, 2016

Copy link
Copy Markdown
Member

This action is completely valid on both Windows and Linux (and OS X)
operating systems; tested with mklink and ln -s respectively.

Note that targets for hard links must exist, thus we check specifically
for symbolic links.

Both the path globber in session state and the New-Item implementation
needed to be fixed to allow the target not to exist.

Resolves #801.

This change is Reviewable

@andyleejordan
Allow creation of symlink to non-existent target
3a43126 
This action is completely valid on both Windows and Linux (and OS X)
operating systems; tested with `mklink` and `ln -s` respectively.

Note that targets for hard links must exist, thus we check specifically
for symbolic links.

Both the path globber in session state and the `New-Item` implementation
needed to be fixed to allow the target not to exist.

Resolves #801.

Also enable symbolic link tests on Windows.
@andyleejordan

Andy Jordan (andyleejordan) commented Apr 10, 2016

Copy link
Copy Markdown
Member Author

FYI Lee Holmes (@LeeHolmes)

@andyleejordan

Andy Jordan (andyleejordan) commented Apr 10, 2016

Copy link
Copy Markdown
Member Author

Merging because I need this to make packages.

@andyleejordan
Andy Jordan (andyleejordan) deleted the andschwa/symbolic-target branch April 10, 2016 01:55
@vors

sergei (vors) commented Apr 11, 2016

Copy link
Copy Markdown
Collaborator

Review status: 0 of 3 files reviewed at latest revision, 3 unresolved discussions.

test/powershell/New-Item.Tests.ps1, line 119 [r1] (raw file):
Please add a symlink test for a directory.

  1. Create non-existing
  2. Test ls on it
  3. Create matching dir
  4. Test ls again

test/powershell/New-Item.Tests.ps1, line 122 [r1] (raw file):
not sure that this is the right behavior

test/powershell/New-Item.Tests.ps1, line 128 [r1] (raw file):
How does somebody can figure out that this symlink is broken?

Comments from Reviewable

@andyleejordan

Andy Jordan (andyleejordan) commented Apr 11, 2016

Copy link
Copy Markdown
Member Author

Review status: 0 of 3 files reviewed at latest revision, 3 unresolved discussions.

test/powershell/New-Item.Tests.ps1, line 119 [r1] (raw file):
Yeah that would be good to have. I'll add it to the backlog.

test/powershell/New-Item.Tests.ps1, line 122 [r1] (raw file):
This is the right behavior; the link file exists. If it didn't, this PR would be wrong.

test/powershell/New-Item.Tests.ps1, line 128 [r1] (raw file):
By testing the existence of its Target, as done here.

Comments from Reviewable

@andyleejordan Andy Jordan (andyleejordan) mentioned this pull request Jan 5, 2017
Closed
@mklement0 Michael Klement (mklement0) mentioned this pull request Apr 7, 2017
Closed
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
@andyleejordan
Merge pull request PowerShell#804 from PowerShell/andschwa/symbolic-t…
f8483f3 
…arget

Allow creation of symlink to non-existent target
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix creation of symbolic links to non-existing files

2 participants

@andyleejordan @vors