Closed Bug 1563221 Opened 4 years ago Closed 3 months ago

Remove older unused unix from NSS source code.

Categories

(NSS :: Build, task, P1)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Usul, Unassigned)

References

Details

(Whiteboard: [nss-qm])

Attachments

(4 files)

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

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

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

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

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

Type: defect → task
Status: NEW → ASSIGNED
Priority: -- → P1

(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.

Whiteboard: [nss-qm]

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.

Assignee: ludovic → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(bbeurdouche)

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.

Flags: needinfo?(ludovic)

I redirected the review to nss reviewers

Flags: needinfo?(ludovic)
You need to log in before you can comment on or make changes to this bug.