Closed
Bug 906656
Opened 11 years ago
Closed 11 years ago
Get a copy of screenresolution in a Mozilla-controlled repo, plus tarballs
Categories
(Infrastructure & Operations :: RelOps: Puppet, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dustin, Unassigned)
Details
Attachments
(1 file)
3.51 KB,
patch
|
armenzg
:
review+
dustin
:
checked-in+
|
Details | Diff | Splinter Review |
We currently build screenresolution 1.5 and 1.6 for different versions of OS X.
We do so by cloning git://github.com/jhford/screenresolution.git. This has two problems:
1. It's a dependency on an external site to maintain the source code
2. The repo's controlled by a single person
We should clone this repo to the mozilla org on github, or to git.m.o. We should also keep tarballs of the source for both 1.5 and 1.6 in the puppetagain repos.
Reporter | ||
Comment 1•11 years ago
|
||
Hal, can you create that repo for me? The repo should be a fork of jhford's. If access is granted to groups, please grant it to IT_Operations (which seems to be the only group I can see) and whatever other groups you prefer.
Flags: needinfo?(hwine)
Reporter | ||
Comment 2•11 years ago
|
||
$ git archive --prefix=screenresolution-1.5 1.5 | gzip > screenresolution-1.5.tar.gz
$ git archive --prefix=screenresolution-1.6 f2c404db62629f504ee56913e59a91470f1d7326 | gzip > screenresolution-1.6.tar.gz
We'll need to tag f2c404db as 1.6 once we've forked it, or perhaps jhford can do so before.
Reporter | ||
Comment 3•11 years ago
|
||
Sorry, those --prefix options should have trailing /'s
Reporter | ||
Comment 4•11 years ago
|
||
Armen, how does this look?
Attachment #792215 -
Flags: review?(armenzg)
Comment 5•11 years ago
|
||
Comment on attachment 792215 [details] [diff] [review]
bug906656.patch
Review of attachment 792215 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm!
Attachment #792215 -
Flags: review?(armenzg) → review+
Reporter | ||
Updated•11 years ago
|
Attachment #792215 -
Flags: checked-in+
Reporter | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 6•11 years ago
|
||
Backed out due to bug 906782
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to Dustin J. Mitchell [:dustin] from comment #1)
> Hal, can you create that repo for me? The repo should be a fork of
> jhford's. If access is granted to groups, please grant it to IT_Operations
> (which seems to be the only group I can see) and whatever other groups you
> prefer.
We prefer not to do things that way (https://wiki.mozilla.org/Github)
:jhford -- the "simplest" way to do this would be for you to transfer ownership to me, then I can transfer it into the Mozilla organization. Does that work for you?
Flags: needinfo?(jhford)
Reporter | ||
Comment 9•11 years ago
|
||
John, ping? I can't add a second needsinfo flag for you, can I (/me tries..)
Assignee: dustin → jhford
Reporter | ||
Updated•11 years ago
|
Flags: needinfo?(jhford)
Reporter | ||
Updated•11 years ago
|
Flags: needinfo?(jhford)
Comment 10•11 years ago
|
||
I'm happy with it as is. Please feel free to fork it into whichever organization you'd like it to be in.
Flags: needinfo?(jhford)
Reporter | ||
Comment 11•11 years ago
|
||
Hal, do you want to set up a new mozilla repo, then? I (if I have perms) or you can push the contents of john's into it.
Assignee: jhford → hwine
Comment 12•11 years ago
|
||
I forked to https://github.com/mozilla/build-screenresolution and gave push access to the releng and it-operations teams.
Flags: needinfo?(hwine)
Reporter | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•