653934 - Add a comment to point people to original Workspace repo to Scratchpad.js

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Rob Campbell [:rc] (:robcee)]

During the transition from the Workspace extension to the landed Scratchpad code, we've lost the original source history. We should add a link back to the originating repo at:

https://github.com/mozilla/workspace

Comment 1

[Rob Campbell [:rc] (:robcee)]

... add the link in a comment in scratchpad.js or wherever appropriate.

Comment 2

[Shawn Wilsher :sdwilsh]

You can migrate a git repo to hg pretty easily; when you land this on m-c you could just hg transplant all those commits (in fact, you can even pull a git repo with hg on github) and then you lose no history.

Comment 3

[Erik Vold]

I haven't tried it yet, but afaict using git fast-export and the hg FastImportExtension might allow one to migrate git history to hg.

Comment 4

[Rob Campbell [:rc] (:robcee)]

Would that work for code that gets repositioned (essentially moved and/or rewritten) to fit into m-c/browser from extension space?

Comment 5

[Erik Vold]

(In reply to comment #4)
> Would that work for code that gets repositioned (essentially moved and/or
> rewritten) to fit into m-c/browser from extension space?

My thinking was that the import would be done in a branch, we have/create a docking folder, import the extension and it's history into that folder, then write child commits which do the integration. That way the import and integration are separated.

Comment 6

[Rob Campbell [:rc] (:robcee)]

yeah, that would work. Could even hg rename a bunch of the initial files across directories to position them and maintain history.

Hm!

Comment 7

[Rob Campbell [:rc] (:robcee)]

Attachment: scratchpad to workspace comment

Pretty basic comment patch. Let me know if there's anything else this needs or if you can think of additional useful verbeage. Maybe it's sufficient!

Comment 8

[Erik Vold]

(In reply to comment #7)
> Created attachment 529809 [details] [diff] [review] [review]
> scratchpad to workspace comment
> 
> Pretty basic comment patch. Let me know if there's anything else this needs or
> if you can think of additional useful verbeage. Maybe it's sufficient!

Can we mention that the original project was a public domain project?

Comment 9

[Rob Campbell [:rc] (:robcee)]

(In reply to comment #8)
> (In reply to comment #7)
> > Created attachment 529809 [details] [diff] [review] [review] [review]
> > scratchpad to workspace comment
> > 
> > Pretty basic comment patch. Let me know if there's anything else this needs or
> > if you can think of additional useful verbeage. Maybe it's sufficient!
> 
> Can we mention that the original project was a public domain project?

I'm not sure. Can we do that, Gerv?

Comment 10

[Gervase Markham [:gerv]]

You can say whatever you like :-) Please try not to get it tangled up with any current licensing statements, though (i.e. put it in a separate comment).

Gerv

Comment 11

[Shawn Wilsher :sdwilsh]

Unfortunately, I'm traveling the so the soonest I'll be able to look at this is Monday, May 9th.

Comment 12

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 529809 [details] [diff] [review]
scratchpad to workspace comment

Maybe gavin's got a second to take a look. Also, any feedback regarding Erik's request to include some verbeage around "originally licensed as public domain" would be welcome.

Comment 13

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 529809 [details] [diff] [review]
scratchpad to workspace comment

Comments are easy to review! You can say what you want as long as it's true!

Comment 14

[Rob Campbell [:rc] (:robcee)]

Attachment: [checked-in][in-devtools] scratchpad to workspace comment 2

Added the relicensing comment and a reference back to this bug.

Comment 15

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 530088 [details] [diff] [review]
[checked-in][in-devtools] scratchpad to workspace comment 2

http://hg.mozilla.org/projects/devtools/rev/3f6040a4e1c8

Comment 16

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 530088 [details] [diff] [review]
[checked-in][in-devtools] scratchpad to workspace comment 2

http://hg.mozilla.org/mozilla-central/rev/3f6040a4e1c8