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

DietPi-Software | ownCloud: Fix Buster installation and cleanup #1325

Merged
merged 1 commit into from
Dec 20, 2017

Conversation

MichaIng
Copy link
Owner

https://github.com/Fourdee/DietPi/issues/1286

  • Clearer separation between Buster and pre-buster steps.
  • Merging of pre-buster URLs.

@MichaIng MichaIng requested a review from Fourdee December 19, 2017 22:02
@MichaIng MichaIng self-assigned this Dec 19, 2017
@MichaIng MichaIng added this to the v160 milestone Dec 19, 2017
@MichaIng
Copy link
Owner Author

MichaIng commented Dec 19, 2017

@Fourdee
New PR organization? So I don't need my own fork any more and can already give labels, that's good 😄.

Self-reminder: Give meaningful names to branches.

@Fourdee
Copy link
Collaborator

Fourdee commented Dec 20, 2017

@MichaIng

New PR organization? So I don't need my own fork any more and can already give labels, that's good 😄.

Yep, with all your contributions to the project, I thought it was time we officially gave you Collaborator status 👍 🥇 Congrats :)

I've disabled raw commits to branch for Collaborators, only myself can do this. PR's must also be merged by me, only reason being it forces me to check before merging 😄

Copy link
Collaborator

@Fourdee Fourdee 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, great work @MichaIng 👍

INSTALL_URL_ADDRESS='https://download.owncloud.org/community/owncloud-latest.zip'

fi
INSTALL_URL_ADDRESS="https://download.owncloud.org/download/repositories/stable/Debian_$(( $DISTRO + 5 )).0/Release.key"
Copy link
Collaborator

Choose a reason for hiding this comment

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

$(( $DISTRO + 5 )) Smart 👍

@Fourdee Fourdee merged commit ea3c48f into testing Dec 20, 2017
@MichaIng MichaIng deleted the MichaIng-patch-1 branch December 20, 2017 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants