1050131 - Write a script to auto-extract from mozilla-central the standalone content into the loop-client git repo

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect, P1)

Description

[Mark Banner (:standard8)]

In bug 1042528 we're defining the initial release process for the standalone content/link-clicker UI web parts of Loop.

For ease of releasing to services, we're extracting the code to a git repository from the authoritative mercurial repository, mozilla-central.

Initial requirements I'm setting myself on the script:

- Extract changesets since the last update
- For each changeset that changes the Loop standalone content, create a new changeset in the git-repository with the changes
- Translate the file structure into one that is suitable for releasing the standalone content (using the structure we have today)
- Enable the tests to be run successfully in the loop-client repository with the new structure

The initial aim is to make this suitable for running in a cron job somewhere.

Comment 1

[Mark Banner (:standard8)]

Attachment: WIP extraction script

This is the WIP I have already, it is still in work, so far it does the basic extraction, but doesn't pull the latest versions of repos, nor does it fix the tests yet.

Comment 2

[Mark Banner (:standard8)]

Attachment: Link to Github pull-request: https://github.com/mozilla/loop-client/pull/38

This implements the script.

Remy: Can you review this from a python/services perspective? 

Dan: Can you check that this looks reasonable, and we're happy with the output and the changeset annotations? There's an example output here:

https://github.com/Standard8/loop-client/compare/1050131-auto-extract

The script is based on MercurialAPI and GitPython. I tried using GitPython all the way through it, but it really was difficult and the documentation is poor.

I suspect with more time, we might be able to get GitPython working properly, but given that what we've now got works, the time spent versus correctness trade-off means I don't think its worth spending extra time.

The patch achieves everything in comment 0, and we can at least start up the local server, have it run correctly, and also load the test files with them succeeding.

Hence this is a good place to start any enhancements from, and means we can get the auto-merge running on a regular basis.

Comment 3

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 8469261 [details] [review]
Link to Github pull-request: https://github.com/mozilla/loop-client/pull/38

r=dmose, with comments in the github PR appropriately addressed.

Comment 4

[Mark Banner (:standard8)]

Addressed comments and merged:

https://github.com/mozilla/loop-client/commit/5487e1138ca10d5b79322141df22a9ce2a8f7f7a
https://github.com/mozilla/loop-client/commit/9388311a89639bf29a1e2ff2441803924a795411

Comment 5

[u279076]

Untracking for QA. Please needinfo me to request testing.