Skip to content

Side-by-side Unix packages - #1717

Merged
Andy Jordan (andyleejordan) merged 7 commits into
PowerShell:masterfrom
andyleejordan:sxs
Aug 11, 2016
Merged

Andy Jordan (andyleejordan) merged 7 commits into
PowerShell:masterfrom
andyleejordan:sxs

Conversation

@andyleejordan

@andyleejordan Andy Jordan (andyleejordan) commented Aug 11, 2016 •

Copy link
Copy Markdown
Member

First go at implementing the side-by-side Unix package design detailed here. I've tested and it's working. I still need to update a bunch of documentation regarding this, but wanted a review.

Please take a look and test Raghu Shantha (@raghushantha).

@raghushantha

Copy link
Copy Markdown
Member

Andy Jordan (@andschwa) Can you include license.txt as part of the package?

Joey Aiello (@joeyaiello) Do we need to include ThirdPartyNotices.txt in the package?

@andyleejordan

Copy link
Copy Markdown
Member Author

Raghu Shantha (@raghushantha) Per 4eea68f the license is included in every copy of the build (and so included int he package).

@raghushantha

Copy link
Copy Markdown
Member

Andy Jordan (@andschwa) Great.

Rather than including it only at packaging, we should include this
license as part of `dotnet build` and `dotnet publish`.
Moved Unix package code into New-UnixPackage, switching on $Types to
call the appropriate function given the list of types.

Also fixed bug with chmod on a symlink; it was unnecessary.
The DLLs should not be marked executable on Unix.

Also remove deb-build-depends as the `dotnet` package is not yet
universally available.
@andyleejordan

Copy link
Copy Markdown
Member Author

Now also resolves #1433.

@joeyaiello

Copy link
Copy Markdown
Contributor

License needs to be in the repo, in the EULA in the MSI installer, and in the root of every package. Binary dependencies need to be in the MSI installer, and the root of every package. Third party notices (the ones applicable to source code) need to be in the root of the repository.

Make sense?

@andyleejordan

Copy link
Copy Markdown
Member Author

Joey Aiello (@joeyaiello) for binary dependencies, can you give me a document for that?

@joeyaiello

Copy link
Copy Markdown
Contributor

Andy Jordan (@andschwa) we don't have it yet. When I do, you guys will be the first to know

@TravisEz13 Travis Plunk (TravisEz13) added the WG-Maintainers-Build specific to affecting the build label Aug 11, 2016
@andyleejordan

Copy link
Copy Markdown
Member Author

No worries. Raghu Shantha (@raghushantha), Joey Aiello (@joeyaiello) can I get a sign off?

@andyleejordan

Copy link
Copy Markdown
Member Author

16.04 package built and tested.

@Francisco-Gamino

Francisco Gamino (Francisco-Gamino) commented Aug 11, 2016 •

Copy link
Copy Markdown
Contributor

Andy Jordan (@andschwa) I tested this on Ubuntu 16.04, and it works like a charm!

@raghushantha

Copy link
Copy Markdown
Member

:shipit:

@andyleejordan

Copy link
Copy Markdown
Member Author

Yay!

@BrucePay

Copy link
Copy Markdown
Collaborator

For what it's worth, I tested both the package and also building on 16.04. Everything worked fine,

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WG-Maintainers-Build specific to affecting the build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants