1563221 - Remove older unused unix from NSS source code.

Status: RESOLVED FIXED

(NSS :: Build, task, P1)

Description

[Ludovic Hirlimann [:Usul]]

I've cleaned up NSPR in bug 1535665 - now following up with NSS.

Comment 1

[Ludovic Hirlimann [:Usul]]

Attachment: Bug 1563221 remove support for older unix in NSS part 1 OSF r?kaie

Comment 2

[Ludovic Hirlimann [:Usul]]

Attachment: Bug 1563221 remove support for older unix in NSS part 2 DGUX r?kaie

Comment 3

[Ludovic Hirlimann [:Usul]]

Attachment: Bug 1563221 remove older unix support in NSS part 3 Irix r?kaie

Comment 4

[Ludovic Hirlimann [:Usul]]

found only tests for Symbian so left them in the source code.

Comment 5

[Kai Engert [:KaiE:]]

Martin, Daiki, Bob, are you OK removing support for those older platforms?

Comment 6

[Martin Thomson [:mt:]]

I've no real objection to doing so, but I probably am not in the best position to know what remains relevant.

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug is a task, but please change it back in case of error.

Comment 8

[Ludovic Hirlimann [:Usul]]

Attachment: Bug 1563221 remove older oses that are unused part3/ BeOS r?kaie

Comment 9

[Kai Engert [:KaiE:]]

(In reply to Martin Thomson [:mt:] from comment #6)

I've no real objection to doing so, but I probably am not in the best position to know what remains relevant.

Support for those platforms has recently been removed from NSPR, so they can no longer be used, see the dependent bug.

Ludo, I think it's better to ask the active NSS maintainers to review the removal patch.

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last months and this bug has priority 'P1'.
:beurdouche, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 11

[BugBot [:suhaib / :marco/ :calixte]]

The following patches are waiting for review from an inactive reviewer:

ID	Title	Author	Reviewer Status
D36755	Bug 1563221 remove support for older unix in NSS part 1 OSF r?kaie	Usul	kaie: Resigned from review
D36756	Bug 1563221 remove support for older unix in NSS part 2 DGUX r?kaie	Usul	kaie: Resigned from review
D36757	Bug 1563221 remove older unix support in NSS part 3 Irix r?kaie	Usul	kaie: Resigned from review
D37215	Bug 1563221 remove older oses that are unused part3/ BeOS r?kaie	Usul	kaie: Resigned from review

:Usul, could you please find another reviewer or abandon the patch if it is no longer relevant?

For more information, please visit auto_nag documentation.

Comment 12

[Sylvestre Ledru [:Sylvestre]]

I redirected the review to nss reviewers

Comment 13

[John Schanck [:jschanck]]

https://hg.mozilla.org/projects/nss/rev/17f9365a7a1d6554be1b31af88f753842fc49b7a
https://hg.mozilla.org/projects/nss/rev/b066df5e9148958e93fa804f8b7688ae897d9c77
https://hg.mozilla.org/projects/nss/rev/ee2e9b06b590e75bc79154fbfacab20a51135655
https://hg.mozilla.org/projects/nss/rev/e0b144ea73b75cc3b9c319dcd2217f09fe6c4584