Skip to content
This repository was archived by the owner on Aug 9, 2024. It is now read-only.

Conversation

@claudiouzelac
Copy link

@claudiouzelac claudiouzelac commented May 13, 2020

This pull request (PR) represents the patches applied at:

https://github.com/mozilla-it/wiki.mozilla.org/tree/master/patches/extensions/Bugzilla

Furthermore, it represents the PR that has been open for 3 years here:

mozilla#97

This change is Reviewable

@ziegeer
Copy link

ziegeer commented May 13, 2020

Oh sweet hornet's nests, do you know if the code before this PR matches what we are running? Has this been tested on stage (or is that the next step?) When this broke before, people came out of the woodwork.

@claudiouzelac
Copy link
Author

I am going to test it locally in docker after getting this back and then promote to staging afterwards.

To confirm this code matches, it matches the current Wiki's patch directory here. The process by which the former Wiki used it was pull down master by .gitmodules:

[submodule "extensions/Bugzilla"]
	path = extensions/Bugzilla
	url = https://github.com/mozilla/mediawiki-bugzilla.git
	branch = master
	ignore = untracked

update.sh applies the patches to both extensions affected:

	patches=`cd patches; find . -type f -name "*.patch"`
if [ -n "$patches" ]; then
    echo
    echo "applying local patches"
    for patch in ; do
        patchdir=`dirname $patch`
        pwd=`pwd`
        cd $patchdir && echo in $patchdir
        git am $pwd/$patch
        cd $pwd
    done
fi

The other update.sh is symlinked to the root directory that has applied patches to deploy from.

Oh sweet hornet's nests, do you know if the code before this PR matches what we are running? Has this been tested on stage (or is that the next step?) When this broke before, people came out of the woodwork.

@ziegeer
Copy link

ziegeer commented May 13, 2020

Great thanks!

@claudiouzelac claudiouzelac merged commit 35eab15 into mozilla-it:master Aug 1, 2020
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.

3 participants