Closed Bug 968259 Opened 12 years ago Closed 12 years ago

Move bzexport into version-control-tools repository

Categories

(Developer Services :: Mercurial: bzexport, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla30

People

(Reporter: gps, Assigned: gps)

References

Details

Attachments

(3 files)

Bug 965330 created the hgcustom/version-control-tools repo. Let's put something in it. I say bzexport is a good first repo. Proposal: 1) Run `hg convert` to dump the full history of https://hg.mozilla.org/users/tmielczarek_mozilla.com/bzexport into hgext/bzexport (or other bikeshedded path) 2) Update `mach mercurial-setup` to reference the new repo 3) Update the existing repo/extension to print an error message when loaded, directing them to the new location. Does this sound reasonable?
Flags: needinfo?(ted)
Flags: needinfo?(sphink)
Fine with me. We should make this widely known so that users of bzexport can convert. Possibly even add a nag output to the existing bzexport repo for people who update from there.
Flags: needinfo?(ted)
No objection here.
Flags: needinfo?(sphink)
I performed `hg convert` and moved the full history of bzexport into hgext/bzexport. Now for the non-trivial part.
The extension now aborts at load time. The error message and documentation references the new location. This shouldn't be pushed until mach mercurial-setup is updated to pull the new repo.
Attachment #8371034 - Flags: review?(ted)
Assignee: nobody → gps
Status: NEW → ASSIGNED
Wow, Splinter really blows at formatting the patch I just uploaded.
bzexport is now pulled from its new canonical home. The old location can be cleaned if the user agrees to it. The new location is updated in the hgrc.
Attachment #8371041 - Flags: review?(ted)
Comment on attachment 8371034 [details] [diff] [review] Redirect content to new repository location Might want to mention that you can `mach mercurial-setup` to get this installed.
Attachment #8371034 - Flags: review?(ted) → review+
Attachment #8371041 - Flags: review?(ted) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
Blocks: 970233
Blocks: 968958
No longer blocks: 970233
Depends on: 970233
Trivial patch to make the dummy extension non-fatal.
Attachment #8378673 - Flags: review?(ted)
Comment on attachment 8378673 [details] [diff] [review] Make redirect non-fatal; Review of attachment 8378673 [details] [diff] [review]: ----------------------------------------------------------------- You could have just pushed this without review. :-P
Attachment #8378673 - Flags: review?(ted) → review+
Product: Other Applications → Developer Services
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: