Closed Bug 653765 Opened 8 years ago Closed 8 years ago

Remove dead WINCE support from SeaMonkey

Categories

(SeaMonkey :: Build Config, defect, trivial)

All
Windows CE
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sgautherie, Assigned: sgautherie)

References

()

Details

(Whiteboard: [fixed in SM 2.4])

Attachments

(2 files)

(Core) Bug 652445 removed this feature.
Flags: in-testsuite-
Attachment #529138 - Flags: review?(bugspam.Callek)
Attachment #529139 - Flags: review?(bugspam.Callek)
This is safe on mozilla5 too, as SeaMonkey never got to the point of actually supporting WinCE, afaik.
Blocks: 614720
Summary: Remove dead MOZ_FASTSTART support from SeaMonkey → Remove dead WINCE support from SeaMonkey
Comment on attachment 529138 [details] [diff] [review]
(Av1) MOZ_FASTSTART
[Checked in: Comment 5]

The approval is only for the suite/* parts, TB-drivers would need to approve the other parts. I'm not sure if they are worth porting to that tree though.
Attachment #529138 - Flags: review?(bugspam.Callek)
Attachment #529138 - Flags: review+
Attachment #529138 - Flags: approval-comm-aurora+
Attachment #529139 - Flags: review?(bugspam.Callek)
Attachment #529139 - Flags: review+
Attachment #529139 - Flags: approval-comm-aurora+
Comment on attachment 529138 [details] [diff] [review]
(Av1) MOZ_FASTSTART
[Checked in: Comment 5]

http://hg.mozilla.org/comm-central/rev/41df0c16c211
Attachment #529138 - Attachment description: (Av1) MOZ_FASTSTART → (Av1) MOZ_FASTSTART [Checked in: Comment 5]
Comment on attachment 529139 [details] [diff] [review]
(Bv1) WINCE
[Checked in: Comment 6]

http://hg.mozilla.org/comm-central/rev/c6bf2e2ff841
Attachment #529139 - Attachment description: (Bv1) WINCE → (Bv1) WINCE [Checked in: Comment 6]
(In reply to comment #4)
> The approval is only for the suite/* parts, TB-drivers would need to approve
> the other parts. I'm not sure if they are worth porting to that tree though.

I'm unfamiliar with comm‑aurora.
I assume these 2 patches would apply but are not "required" there.
(Anyone) Feel free to land them there if you want to. I don't plan to.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.3
Depends on: 651622
(In reply to comment #7)
> (In reply to comment #4)
> > The approval is only for the suite/* parts, TB-drivers would need to approve
> > the other parts. I'm not sure if they are worth porting to that tree though.
> 
> I'm unfamiliar with comm‑aurora.
> I assume these 2 patches would apply but are not "required" there.

Correct.

> (Anyone) Feel free to land them there if you want to. I don't plan to.

Somehow I thought I read/saw you ask for these to land there, guess not. if you don't plan to land them, I see no reason they should land then.
(In reply to comment #8)

> > I'm unfamiliar with comm‑aurora.

Ah, comm-aurora is currently related to "moz6".

> Somehow I thought I read/saw you ask for these to land there, guess not.

I did mean this to be applied to "moz6", but that was before comm-aurora existed: too bad/late for this dead code removal...
Whiteboard: [fixed in SM 2.4]
Target Milestone: seamonkey2.3 → ---
You need to log in before you can comment on or make changes to this bug.