Switch from PRBool to bool and replace PR_TRUE/PR_FALSE with true/false in comm-central

RESOLVED FIXED in Thunderbird 12.0

Status

MailNews Core
Backend
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Philip Chee, Assigned: standard8)

Tracking

Trunk
Thunderbird 12.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug])

Attachments

(3 attachments)

(Reporter)

Description

6 years ago
q.v. Mozilla Central bugs:
Bug 690892 - replace PR_TRUE/PR_FALSE with true/false in mozilla-central.
Bug 675553 - Switch from PRBool to bool on a CLOSED TREE.
(Assignee)

Comment 1

6 years ago
I think considering this is just a script run against the repo, we should just do all of comm-central in one changeset and be done with it.
Component: General → Backend
Product: SeaMonkey → MailNews Core
QA Contact: general → backend
Summary: Switch from PRBool to bool and replace PR_TRUE/PR_FALSE with true/false in Suite → Switch from PRBool to bool and replace PR_TRUE/PR_FALSE with true/false in comm-central
(In reply to Mark Banner (:standard8) from comment #1)
> I think considering this is just a script run against the repo, we should
> just do all of comm-central in one changeset and be done with it.

Sounds Good. You can do this if you like, with post-landing review from me if you want. And promise to spot-check first :-) (so hopefully we don't have in-tree-bustage)

(I just want to wait until the SeaMonkey buildbot master comes back up before landing it)
(Assignee)

Comment 3

6 years ago
Well, I'd at least expect a run through try server first.
(Assignee)

Comment 4

6 years ago
Created attachment 584783 [details]
PRBool replacement script for sed.
Assignee: nobody → mbanner
Status: NEW → ASSIGNED
(Assignee)

Comment 5

6 years ago
Created attachment 584785 [details]
PR_TRUE/FALSE replacement script for sed
(Assignee)

Comment 6

6 years ago
Created attachment 584790 [details]
find & sed script for replacements

I think whilst we're in this natural quiet period we should just get this done - afaik no big patches are pending, and they can always be patched - see the scripts on the dependent bugs.

These scripts are for OS X, replace <script> in this attachment with the name of the sed script from either of the other attachments.

I've pushed this to try as:

http://hg.mozilla.org/try-comm-central/rev/c60b80c3f05e

http://build.mozillamessaging.com/tinderboxpushlog/?tree=ThunderbirdTry&rev=c60b80c3f05e

If the builds are successful and pass tests, I'm proposing to push this to comm-central as soon as I can - I'll do a post to the newsgroups as well.
Attachment #584790 - Flags: review?(dbienvenu)
(Assignee)

Updated

6 years ago
Attachment #584783 - Attachment is patch: false
In reply to comment #6:
IIUC, this covers Thunderbird and MailNews Core. What about SeaMonkey code not shared with Fx & Tb?
- this bug or another bug?
- covered yes/no by the sed scripts?
- covered yes/no by the try builds?
- tested yes/no after building?

Comment 8

6 years ago
Comment on attachment 584790 [details]
find & sed script for replacements

I think this is just the find script, but it looks OK.
Attachment #584790 - Flags: review?(dbienvenu) → review+
(Assignee)

Comment 9

6 years ago
(In reply to David :Bienvenu from comment #8)
> I think this is just the find script, but it looks OK.

Yes, see the other two attachments (sorry, I wasn't caring too much about individual reviews on each).

(In reply to Tony Mechelynck [:tonymec] from comment #7)
> In reply to comment #6:
> IIUC, this covers Thunderbird and MailNews Core. What about SeaMonkey code
> not shared with Fx & Tb?

This covers all of comm-central code apart from ldap/sdks/c-sdk which is imported from an external location, and would be up to the maintainers to change if they wanted to.
(Assignee)

Comment 10

6 years ago
I landed this earlier today, everything seems fine still:

http://hg.mozilla.org/comm-central/rev/f9d611e3d0a5
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 12.0

Updated

5 years ago
Duplicate of this bug: 108584
You need to log in before you can comment on or make changes to this bug.