Skip to content
This repository has been archived by the owner on Sep 23, 2023. It is now read-only.

Allow patches to be updated #270

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Allow patches to be updated #270

wants to merge 2 commits into from

Conversation

edg2s
Copy link
Collaborator

@edg2s edg2s commented Mar 6, 2021

Fixes #54

@edg2s
Copy link
Collaborator Author

edg2s commented Mar 8, 2021

TODO:

  • Run composer update
  • Run maintenance script
  • Improve UI

@MatmaRex
Copy link
Owner

MatmaRex commented Mar 9, 2021

This will overwrite random files belonging to other wikis, because the deduplication uses hardlinks.

@edg2s
Copy link
Collaborator Author

edg2s commented Mar 9, 2021

Is there a way to turn off the duplication after it's been done?

@MatmaRex
Copy link
Owner

MatmaRex commented Mar 9, 2021 via email

@edg2s
Copy link
Collaborator Author

edg2s commented Feb 27, 2022

You could just copy the whole directory. There might be another way, but this will definitely work.

Done. Copying the files is a fairly quick operation.

@edg2s edg2s changed the title WIP Allow patches to be updated Allow patches to be updated Feb 28, 2022
@edg2s edg2s force-pushed the update branch 2 times, most recently from eb850d3 to d2422d3 Compare February 28, 2022 17:01
@edg2s edg2s requested a review from MatmaRex February 28, 2022 17:25
@edg2s edg2s force-pushed the update branch 2 times, most recently from b8585b1 to 914b903 Compare February 28, 2022 17:32
new/unlink.sh Outdated

# Copy all files so they are no longer hard-link duplicates of other
# files on the system (reverse of deduplicate.js)
cp -r $PATCHDEMO/wikis/$NAME/w $PATCHDEMO/wikis/$NAME/w-cloned
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Offline code review from @Tchanders suggested this could fail if rdfind is running while we are updating the new wiki.

My initial though would be to leave the wiki in w-cloned while updating, and modify our rdfind cron job to exclude folders called w-cloned.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed (used w-updating)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update wikis with newer patchset, retaining database
2 participants