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

Generate AppImage #2027

Closed
wants to merge 25 commits into from
Closed

Generate AppImage #2027

wants to merge 25 commits into from

Conversation

probonopd
Copy link
Contributor

@probonopd probonopd commented Aug 23, 2016

Closes #2024

AppImage is a packaging format for Linux that lets users download one single file, set the executable bit and run the application (here: PowerShell). Unlike other solutions, no special runtime infrastructure must be installed on the target system(s).

This pull request, when merged, will package PowerShell as an AppImage and upload it.

@msftclas
Copy link

Hi @probonopd, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@msftclas
Copy link

@probonopd, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, MSBOT;

@probonopd
Copy link
Contributor Author

Here is the log of a build in which the AppImage actually gets generated (it does not in PRs, as do the other packages): https://travis-ci.org/probonopd/PowerShell/builds/154322840

@daxian-dbw daxian-dbw added the Area-Maintainers-Build specific to affecting the build label Aug 23, 2016
@@ -2,14 +2,15 @@ Package installation instructions
=================================

Supports [Ubuntu 14.04][u14], [Ubuntu 16.04][u16],
[CentOS 7][cos], and [OS X 10.11][osx].
[CentOS 7][cos], [most Linux distributions (AppImage)][lai], and [OS X 10.11][osx].
Copy link
Member

@TravisEz13 TravisEz13 Aug 23, 2016

Choose a reason for hiding this comment

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

Can we say many instead of most? And add a link to AppImage for more info. #Resolved

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, done

@TravisEz13
Copy link
Member

TravisEz13 commented Aug 23, 2016

Can you do an interactive rebase so that there is one change per file (or less?) #WontFix

@andyleejordan
Copy link
Member

It doesn't appear to be a problem related to the AppImage. I tested both saving and restoring the location; and not doing the AppImage generation until after the Pester tests. I'm testing a rebase now.

andyleejordan and others added 5 commits August 26, 2016 11:42
As it works independently of New-UnixPackage,
I don't think it should be run there.
Also put version back to 6.0.0-alpha.9 (- instead of .).
Removed $LOWERAPP as it was no longer needed.
The previous failure we saw was because the most recent annotated tag
did not exist in the previous 50 commits. In my experience we've gone
about ~300 commits between tags, so I believe 1000 to be safe, but know
100 will not be.
Suggested changes for AppImage
andyleejordan pushed a commit to andyleejordan/PowerShell that referenced this pull request Aug 26, 2016
andyleejordan pushed a commit to andyleejordan/PowerShell that referenced this pull request Aug 26, 2016
andyleejordan pushed a commit to andyleejordan/PowerShell that referenced this pull request Aug 26, 2016
andyleejordan pushed a commit to andyleejordan/PowerShell that referenced this pull request Aug 26, 2016
andyleejordan pushed a commit to andyleejordan/PowerShell that referenced this pull request Aug 26, 2016
andyleejordan pushed a commit to andyleejordan/PowerShell that referenced this pull request Aug 26, 2016
@andyleejordan andyleejordan mentioned this pull request Aug 26, 2016
@probonopd
Copy link
Contributor Author

Continued in #2082, hence closing here.

@probonopd probonopd closed this Aug 28, 2016
andyleejordan pushed a commit that referenced this pull request Feb 14, 2017
* Use icon from local repository
  #2027 (comment)

* Use the deb that has been generated in this build

* Copyright and license
  #2027 (comment)

* MIT License for appimage.sh
  #2027 (comment)

* Full text of the MIT License is in license_thirdparty_proprietary.txt

* Clarify license and clean up unused code
  #2027 (comment)

* Mark appimage.sh as executable
andyleejordan pushed a commit to andyleejordan/PowerShell that referenced this pull request Feb 17, 2017
* Use icon from local repository
  PowerShell#2027 (comment)

* Use the deb that has been generated in this build

* Copyright and license
  PowerShell#2027 (comment)

* MIT License for appimage.sh
  PowerShell#2027 (comment)

* Full text of the MIT License is in license_thirdparty_proprietary.txt

* Clarify license and clean up unused code
  PowerShell#2027 (comment)

* Mark appimage.sh as executable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Maintainers-Build specific to affecting the build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants