The default bug view has changed. See this FAQ.

Unfork <msgCompSecurityInfo.xul>

RESOLVED FIXED in Thunderbird 11.0

Status

Thunderbird
Security
--
trivial
RESOLVED FIXED
9 years ago
5 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

Trunk
Thunderbird 11.0
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
*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 ?
(Assignee)

Comment 1

9 years ago
Everyone, ping ? (on my 2 questions)
Flags: in-testsuite-
Target Milestone: --- → Thunderbird 3

Comment 2

9 years ago
sorry, I'm not clear - can we completely unfork smime?
(Assignee)

Comment 3

9 years ago
(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.
Component: Build Config → Security
QA Contact: build-config → thunderbird
(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.
Depends on: 442189
(Assignee)

Comment 5

5 years ago
(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.
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
No longer depends on: 442189
Target Milestone: Thunderbird 3 → ---
(Assignee)

Comment 6

5 years ago
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...
Attachment #574606 - Flags: review?(mbanner)
Attachment #574606 - Flags: review?(mbanner) → review+
(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).
(Assignee)

Comment 8

5 years ago
Comment on attachment 574606 [details] [diff] [review]
Unfork (TB) <msgCompSecurityInfo.xul>
[Checked in: Comment 8]

http://hg.mozilla.org/comm-central/rev/478cd65b1e96
Attachment #574606 - Attachment description: Unfork (TB) <msgCompSecurityInfo.xul>. → Unfork (TB) <msgCompSecurityInfo.xul> [Checked in: Comment 8]
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
You need to log in before you can comment on or make changes to this bug.