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.

Sign up for GitHub

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

Jump to bottom

Update pear/archive_tar to 1.3.13 #11740

Merged
merged 1 commit into from Oct 30, 2014
Merged

Update pear/archive_tar to 1.3.13 #11740

merged 1 commit into from Oct 30, 2014

Conversation

DeepDiver1975
Copy link
Member

@MorrisJobke
Copy link
Contributor

Works 👍 (tested with downloading from files app and from shared folder)

@DeepDiver1975
Copy link
Member Author

Works 👍 (tested with downloading from files app and from shared folder)

Installing an app from the appstore which is packaged as tar would be a great test case

@DeepDiver1975
Copy link
Member Author

Do you know of a certain app on the appstore that is packaged as tar?

http://apps.owncloud.com/content/download.php?content=167576&id=1&tan=86187749

@DeepDiver1975
Copy link
Member Author

Please review @icewind1991 @bantu @PVince81 @butonic THX

@DeepDiver1975 DeepDiver1975 mentioned this pull request Oct 28, 2014
Merged
@MorrisJobke
Copy link
Contributor

I wil ltake care of the needed rebase due to also updated 3rdparty submodule

@DeepDiver1975
Copy link
Member Author

thx @MorrisJobke

@MorrisJobke
Copy link
Contributor

Updated the 3rdparty repo and this PR.

@DeepDiver1975
Copy link
Member Author

Updated the 3rdparty repo and this PR.

thx

@MorrisJobke
Copy link
Contributor

Another round of updating 3rdparty and then this will be fixed :)

@MorrisJobke MorrisJobke force-pushed the update-archive-tar branch 2 times, most recently from 68b5c9e to 6f54ade Compare October 29, 2014 13:16
@MorrisJobke
Copy link
Contributor

Updated :)

@ghost
Copy link

ghost commented Oct 29, 2014

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/1538/
🚀 Test PASSed. 🚀

@georgehrke
Copy link
Contributor

👍

@DeepDiver1975
Copy link
Member Author

will rebase ...

@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@ghost
Copy link

ghost commented Oct 30, 2014

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/1677/
🚀 Test PASSed. 🚀

DeepDiver1975 added a commit that referenced this pull request Oct 30, 2014
@DeepDiver1975
Merge pull request #11740 from owncloud/update-archive-tar
9b29353 
Update pear/archive_tar to 1.3.13
@DeepDiver1975 DeepDiver1975 merged commit 9b29353 into master Oct 30, 2014
@DeepDiver1975 DeepDiver1975 deleted the update-archive-tar branch October 30, 2014 16:14
@lock lock bot locked as resolved and limited conversation to collaborators Aug 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
3 - To Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@DeepDiver1975 @MorrisJobke @georgehrke @scrutinizer-notifier