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

mono: relocate special folders #3154

Merged
merged 1 commit into from
Feb 24, 2018

Conversation

ymartin59
Copy link
Contributor

@ymartin59 ymartin59 commented Feb 17, 2018

Motivation: Preserve certificates store from DSM upgrades
Linked issues: #3150

Checklist

  • Build rule all-supported completed successfully (except not supported PPC architectures)
  • Package upgrade completed successfully
  • New installation of package completed successfully

@ymartin59 ymartin59 changed the title mono: redirect special folders mono: relocate special folders Feb 17, 2018

service_postupgrade ()
{
sync_certificates
Copy link
Contributor

Choose a reason for hiding this comment

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

Why twice? Since it's already done during postinst?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right. I just copied it from previous installer.sh

Preserve certificates store from DSM upgrades
Use generic installer script
@ymartin59 ymartin59 force-pushed the mono-relocate-appdata branch from bc48938 to 0045b95 Compare February 19, 2018 21:33
@ymartin59
Copy link
Contributor Author

ymartin59 commented Feb 19, 2018

@Safihre May you please build this fix and submit as test packages, with hope to receive some feedbacks before I merge and publish in repository ? Thank you in advance

@Safihre Safihre force-pushed the mono-relocate-appdata branch from e7bfa8f to 0045b95 Compare February 20, 2018 07:24
@Safihre
Copy link
Contributor

Safihre commented Feb 20, 2018

Building and updated the link in #3138.

@ymartin59
Copy link
Contributor Author

@Safihre Have you received any feedback about this latest mono package version ?

@Safihre
Copy link
Contributor

Safihre commented Feb 23, 2018

Not really, there seem to be 31 downloads:
http://www.safihre.com/sabnzbd/stats.html?repo=safihre/spksrc

@ymartin59 ymartin59 merged commit 8b01fbb into SynoCommunity:master Feb 24, 2018
@ymartin59 ymartin59 deleted the mono-relocate-appdata branch February 24, 2018 08:45
@Safihre
Copy link
Contributor

Safihre commented Feb 24, 2018

Tried to test it on my Xpenology but doing an update caused it to not start anymore (makes sense of course 🤒😂).

@Safihre
Copy link
Contributor

Safihre commented Feb 24, 2018

(DSM update I mean)

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.

2 participants