Skip to content

Create Third Party Notices.txt #603

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

Merged
merged 1 commit into from
Jan 8, 2018
Merged

Conversation

TylerLeonhardt
Copy link
Member

@TylerLeonhardt TylerLeonhardt commented Jan 6, 2018

Copy link
Contributor

@daviwil daviwil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! We should include this .txt file when the PSES module is distributed

@TylerLeonhardt
Copy link
Member Author

@daviwil is that done in an automated way somewhere?

@TylerLeonhardt
Copy link
Member Author

Maybe here? https://github.com/PowerShell/PowerShellEditorServices/blob/master/PowerShellEditorServices.build.ps1#L185

@daviwil
Copy link
Contributor

daviwil commented Jan 6, 2018

Yep, that works


UnixConsoleEcho

Copyright (c) 2017 Patrick Meinecke
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be 2018 or at least 2017-2018? :-)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you need to specify a range, copyright automatically extends quite a few years after initial creation :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough. Also, I've been told by our legal department that Copyright (c) is redundant. You only need one of: Copyright or (c) or ©. RE ranges and copyright update, I was told it should be updated whenever the associated source file (or copyrighted artifact) is updated so in our older code-base you see a combination of comma separate years and ranges. Because of the "copyright extends past the last date for some (large) number of years" we got our legal dept to buy off on us not putting copyright years in our source code. It got to be too much of a PITA to keep it correct and besides, devs often forget to update copyright notices in source code anyway. :-)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty sure he was just copying my notice :P

If it makes anything easier I can just remove the copyright from the license. Pretty sure it was just a part of whatever template I copied it from :)

Either way I can guarantee I'll never update it :P


UnixConsoleEcho

Copyright (c) 2017 Patrick Meinecke
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough. Also, I've been told by our legal department that Copyright (c) is redundant. You only need one of: Copyright or (c) or ©. RE ranges and copyright update, I was told it should be updated whenever the associated source file (or copyrighted artifact) is updated so in our older code-base you see a combination of comma separate years and ranges. Because of the "copyright extends past the last date for some (large) number of years" we got our legal dept to buy off on us not putting copyright years in our source code. It got to be too much of a PITA to keep it correct and besides, devs often forget to update copyright notices in source code anyway. :-)

@rkeithhill
Copy link
Contributor

It's fine. Copyright notices get me going having gone a few rounds with our legal folks. Go ahead and merge it.

@TylerLeonhardt TylerLeonhardt merged commit a11d9c0 into master Jan 8, 2018
@TylerLeonhardt TylerLeonhardt deleted the third-party-notices branch January 8, 2018 17:49
TylerLeonhardt pushed a commit to TylerLeonhardt/PowerShellEditorServices that referenced this pull request Feb 26, 2019
…owerShell#1281)

Fix PowerShell#603 where system has ipv6 addresses disabled.

This is also more consistent with the PSES TCP listener which always
[IPAddress]::Loopback as the IP address.
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.

4 participants