Closed Bug 544913 Opened 11 years ago Closed 11 years ago

Restore NS_PRINTING and use it in packaging

Categories

(MailNews Core :: Build Config, defect)

defect
Not set
trivial

Tracking

(thunderbird3.0 wontfix)

RESOLVED FIXED
Thunderbird 3.1b1
Tracking Status
thunderbird3.0 --- wontfix

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(2 files)

See discussion in bug 513709.
Then port bug 468463 too.
Flags: in-testsuite-
"approval-seamonkey2.0.4=?":
Zero risk, build config only.
Attachment #425833 - Flags: approval-seamonkey2.0.4?
Attachment #425833 - Flags: approval-seamonkey2.0.4? → approval-thunderbird3.0.2?
Attachment #425833 - Flags: approval-thunderbird3.0.2?
Comment on attachment 425826 [details] [diff] [review]
(Av1) Restore NS_PRINTING, Port bug 468463, Use it in packaging
[Checkin: Comment 4]

r+. But I can't vouch for the calendar part. And I forget if I have a right to r+ the mail/ part here. (I'll try and touch base with Mark and Gozer later to see if I can in future patches)
Attachment #425826 - Flags: review?(bugspam.Callek) → review+
Attachment #425826 - Attachment description: (Av1) Restore NS_PRINTING, Port bug 468463, Use it in packaging → (Av1) Restore NS_PRINTING, Port bug 468463, Use it in packaging [Checkin: Comment 4]
Comment on attachment 425826 [details] [diff] [review]
(Av1) Restore NS_PRINTING, Port bug 468463, Use it in packaging
[Checkin: Comment 4]


http://hg.mozilla.org/comm-central/rev/d99505b45b44
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Attachment #425833 - Flags: approval-thunderbird3.0.2?
Comment on attachment 425833 [details] [diff] [review]
(Bv1-191) Use it in packaging, Remove NS_PRINT_PREVIEW

There's no reason to take this on branch. Qt port isn't likely to happen there.
Attachment #425833 - Flags: approval-thunderbird3.0.2? → approval-thunderbird3.0.2-
(In reply to comment #3)
> (From update of attachment 425826 [details] [diff] [review])
> r+. But I can't vouch for the calendar part.

I hope Serge at least touched base with Fallen on this.
(In reply to comment #5)
> There's no reason to take this on branch. Qt port isn't likely to happen there.

Ftr, this patch was not specifically targeted at Qt support.
You need to log in before you can comment on or make changes to this bug.