Last Comment Bug 442838 - Unfork <msgCompSecurityInfo.xul>
: Unfork <msgCompSecurityInfo.xul>
Status: RESOLVED FIXED
:
Product: Thunderbird
Classification: Client Software
Component: Security (show other bugs)
: Trunk
: All All
: -- trivial (vote)
: Thunderbird 11.0
Assigned To: Serge Gautherie (:sgautherie)
:
Mentors:
http://mxr.mozilla.org/comm-central/f...
Depends on: 441342
Blocks:
  Show dependency treegraph
 
Reported: 2008-06-30 21:00 PDT by Serge Gautherie (:sgautherie)
Modified: 2011-12-02 07:58 PST (History)
6 users (show)
bugzillamozillaorg_serge_20140323: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Unfork (TB) <msgCompSecurityInfo.xul> [Checked in: Comment 8] (6.92 KB, patch)
2011-11-15 09:14 PST, Serge Gautherie (:sgautherie)
standard8: review+
Details | Diff | Splinter Review

Description Serge Gautherie (:sgautherie) 2008-06-30 21:00:20 PDT
*1 js: mailnews/, shared :-)

*2 css + 1 dtd + 1 properties: mail/ and suite/, but otherwise identical :-|
 *Would there be a way to share them too, in mailnews/ ?

*1 xul: mail/ and mailnews/ :-(
 *They are (now) identical, except for |s/#/   -/g| for the license. (same text)

<http://mxr.mozilla.org/mozilla/search?string=msgCompSecurityInfo.xul&find=%2Fsmime%2Fjar%5C.mn%24&filter=%5E%5B%5E%5C0%5D*%24&tree=mozilla>
Do we want to preprocess the mailnews/ file, or keep it as it is ?
Comment 1 Serge Gautherie (:sgautherie) 2008-08-20 00:50:24 PDT
Everyone, ping ? (on my 2 questions)
Comment 2 David :Bienvenu 2008-08-20 07:48:36 PDT
sorry, I'm not clear - can we completely unfork smime?
Comment 3 Serge Gautherie (:sgautherie) 2008-08-20 08:07:07 PDT
(In reply to comment #2)

<http://mxr.mozilla.org/comm-central/find?string=%2Fmail%2Fextensions%2Fsmime%2F>
<http://mxr.mozilla.org/comm-central/find?string=%2Fmailnews%2Fextensions%2Fsmime%2Fresources%2Fcontent%2F>
<http://mxr.mozilla.org/comm-central/find?string=%2Fsuite%2F.*%2Fsmime%2F&tree=comm-central>

> sorry, I'm not clear - can we completely unfork smime?

That would be my goal, but currently blocked by bug 442189 at least:
this is why I filed this for this file only atm.
Comment 4 Gary Kwong [:gkw] [:nth10sd] 2009-02-24 04:34:53 PST
(In reply to comment #3)
> That would be my goal, but currently blocked by bug 442189 at least:
> this is why I filed this for this file only atm.

Adding dependency based on comment #3.
Comment 5 Serge Gautherie (:sgautherie) 2011-11-15 08:48:37 PST
(In reply to David :Bienvenu from comment #2)
> sorry, I'm not clear - can we completely unfork smime?

(In reply to Gary Kwong [:gkw, :nth10sd] from comment #4)
> Adding dependency based on comment #3.

Unless there is an explicit objection, waiting to do it all at once (would) probably means "never": let's do this part now.
Comment 6 Serge Gautherie (:sgautherie) 2011-11-15 09:14:20 PST
Created attachment 574606 [details] [diff] [review]
Unfork (TB) <msgCompSecurityInfo.xul>
[Checked in: Comment 8]

(In reply to Serge Gautherie (:sgautherie) from comment #0)

> *2 css + 1 dtd + 1 properties: mail/ and suite/, but otherwise identical :-|
>  *Would there be a way to share them too, in mailnews/ ?

.dtd and .properties files are identical:
do we want to share them?
is there a syntax to use in /mail jar.mn to package them from /mailnews?

2 SM + 3 TB css files are identical:
same questions.

> Do we want to preprocess the mailnews/ file, or keep it as it is ?

I default to no preprocessing...
Comment 7 Mark Banner (:standard8) (afk until 26th July) 2011-12-02 03:40:17 PST
(In reply to Serge Gautherie (:sgautherie) from comment #6)
> > *2 css + 1 dtd + 1 properties: mail/ and suite/, but otherwise identical :-|
> >  *Would there be a way to share them too, in mailnews/ ?
> 
> .dtd and .properties files are identical:
> do we want to share them?
> is there a syntax to use in /mail jar.mn to package them from /mailnews?

The long term solution would be to move all the files into mailnews and set up l10n correctly and share them from there. However, I certainly wouldn't want to do that piecemeal as it just makes things complicated for localisers. Hence I wouldn't want to merge these ones in this patch/bug (and I think there's already a bug out somewhere for merging the files).
Comment 8 Serge Gautherie (:sgautherie) 2011-12-02 07:57:56 PST
Comment on attachment 574606 [details] [diff] [review]
Unfork (TB) <msgCompSecurityInfo.xul>
[Checked in: Comment 8]

http://hg.mozilla.org/comm-central/rev/478cd65b1e96

Note You need to log in before you can comment on or make changes to this bug.