Closed Bug 1551033 Opened 5 years ago Closed 3 years ago

Fork ChatZilla into SeaMonkey

Categories

(SeaMonkey :: Build Config, task)

task
Not set
normal

Tracking

(seamonkey2.53+ fixed, seamonkey2.57esr? affected)

RESOLVED FIXED
seamonkey 2.85
Tracking Status
seamonkey2.53 + fixed
seamonkey2.57esr ? affected

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

Attachments

(3 files)

In order to fold ChatZilla and DOM Inspector into SeaMonkey, the existing files (and preferably histories) need to be pulled into comm-* from their respective repositories but without any branches.
The first step for each would be to clone the repository and run hg convert --filemap to move the files into suite/extensions/[irc|inspector] and strip anything that is not the default branch
The next step would be to bring the information into comm-*.
Questions:
Is there a way of doing this, whilst keeping the history of about 2500 changes, without polluting the recent changelog?
What is the best way of importing the files and history?

Other times we have imported into an area of suite/ we have pulled the complete history in and attached to the tip of the comm repositories. We'd do as two sets, one for ChatZilla and one for DOM Inspector.
Any issues with this being done for comm-central, comm-esr60 and the SM2.53 branch on comm-release?

Flags: needinfo?(rob)
Flags: needinfo?(mkmelin+mozilla)

Why do you want to do it now, after all these years?
And aren't both of those super dead since at least a year or two?

I'm not sure anyone really cares if it's for comm-esr60 or the sm branch only, but I don't think we should add dead code to comm-central.

For how, hg convert indeed. I don't think there is any way of not polluting recent changelogs, at least the ones showing commits based on time.

Flags: needinfo?(rob)
Flags: needinfo?(mkmelin+mozilla)

(In reply to Magnus Melin [:mkmelin] from comment #2)

Why do you want to do it now, after all these years?
And aren't both of those super dead since at least a year or two?

I'm not sure anyone really cares if it's for comm-esr60 or the sm branch only, but I don't think we should add dead code to comm-central.

For how, hg convert indeed. I don't think there is any way of not polluting recent changelogs, at least the ones showing commits based on time.
The current long term plan is to integrate them into SM rather than them be add-ons, but also get localisation working smoothly for them which I think won't happen whilst they're in separate repos and not showing in comm-central

I wonder how useful this is for DOMI as we have a DOM inspection tool in devtools as well (I know, somewhat different, but still similar). For ChatZilla, I see a better case in general.

There are likely some changes coming for l10n and thunderbird (separate repo), so having the code in comm-central won't likely make anything any easier.

I think the better question is, do you actually see any chance for a seamonkey based on anything past 60? I have to say it would appear extremely unlikely you can pull that off from my POV.

Summary: Fork ChatZilla and DOM Inspector into SeaMonkey → Fork ChatZilla into SeaMonkey
Version: SeaMonkey 2.49 Branch → SeaMonkey 2.57 Branch
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED

Changes needed for the move into suite/extensions

Attachment #9201874 - Flags: review?(frgrahl)
Attachment #9201874 - Flags: approval-comm-release?
Attachment #9201874 - Flags: approval-comm-esr60?

Comment on attachment 9201874 [details] [diff] [review]
1551033-suite-extension-2537.patch

LGTM

There is room for improvement in the build files but I would do this in a follow-up. Personally I don't need the history in c-c and would be ok with just importing the current source which should not really add much.

Next step would be to integrate and get rid of the extension.

Attachment #9201874 - Flags: review?(frgrahl)
Attachment #9201874 - Flags: review+
Attachment #9201874 - Flags: approval-comm-release?
Attachment #9201874 - Flags: approval-comm-release+
Attachment #9201874 - Flags: approval-comm-esr60?
Attachment #9201874 - Flags: approval-comm-esr60+

Initial import patch of the cZ source without history. IanN did it. r+ from me.

Attachment #9205504 - Flags: review+

Rebased intial build patch. en-US works here but l10n packaging not. This is expected and goses away after integration the sources later. r+ from me.

Attachment #9205513 - Flags: review+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/c19a0da417e2
Initial import of ChatZilla sources into SeaMonkey. r=frg
https://hg.mozilla.org/comm-central/rev/7517593cc9c5
Fork ChatZilla into SeaMonkey - changes for move. r=frg

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/238eb10d8972019e2275ea3dd412ab28ffbd73a0
Fork ChatZilla into SeaMonkey - changes for move. r=frg a=frg
Target 2.53.7

Target Milestone: Future → seamonkey 2.85
Version: SeaMonkey 2.57 Branch → Trunk
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: