Closed
Bug 286957
Opened 20 years ago
Closed 20 years ago
bump copyright year in browser/app/nsBrowserApp.cpp
Categories
(Firefox :: General, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: chase, Assigned: steffen.wilberg)
Details
(Keywords: fixed-aviary1.0.3)
Attachments
(1 file, 1 obsolete file)
|
698 bytes,
patch
|
dbaron
:
review+
chase
:
approval-aviary1.0.3+
asa
:
approval-aviary1.1a1+
|
Details | Diff | Splinter Review |
The copyright year needs to be bumped in browser/app/nsBrowserApp.cpp.
| Reporter | ||
Comment 1•20 years ago
|
||
Nominating blocking-aviary1.0.3 and blocking-aviary1.1.
Status: NEW → ASSIGNED
Flags: blocking-aviary1.1?
Flags: blocking-aviary1.0.3?
Updated•20 years ago
|
Hardware: PC → All
| Assignee | ||
Comment 3•20 years ago
|
||
Comment on attachment 178088 [details] [diff] [review] patch Always request a review from somebody.
Attachment #178088 -
Flags: review?(dbaron)
Attachment #178088 -
Flags: review?
Attachment #178088 -
Flags: approval-aviary1.0.3?
Comment on attachment 178088 [details] [diff] [review] patch Did you put this in release-checklist? And should it be 2004-2005 instead of just replacing with 2005?
Attachment #178088 -
Flags: review?(dbaron) → review+
| Assignee | ||
Comment 5•20 years ago
|
||
> Did you put this in release-checklist? I did. > And should it be 2004-2005 instead of just replacing with 2005? Hmm, probably. Or 1998-2005, like the about dialog?
That depends what it's describing the copyright of. What is it describing? :-)
| Assignee | ||
Comment 7•20 years ago
|
||
This string is displayed when typing "firefox -v" on the command line: Mozilla Firefox 1.0, Copyright (c) 2004 mozilla.org I'd say this is describing Firefox, which didn't exist back in 1998. The first release was in 2004, so it should be (c) 2004-2005. On the other hand, the About dialog in both Firefox and Thunderbird display (c) 1998-2005. This should probably be consistent. But im not an expert in copyright law, so I don't know which one to pick. CC'ing Mitchell and Gerv.
Comment 8•20 years ago
|
||
Comment on attachment 178088 [details] [diff] [review] patch not making 1.0.3, try later. I don't think this copyright is correct. It should match the about screen one way or another. "Firefox" may be recent, but the copyrighted code includes bits that have been around a lot longer.
Attachment #178088 -
Flags: superreview-
Attachment #178088 -
Flags: approval-aviary1.0.4?
Attachment #178088 -
Flags: approval-aviary1.0.3?
Attachment #178088 -
Flags: approval-aviary1.0.3-
Updated•20 years ago
|
Flags: blocking-aviary1.0.4?
Updated•20 years ago
|
Flags: blocking-aviary1.1? → blocking-aviary1.1+
| Assignee | ||
Comment 9•20 years ago
|
||
Assignee: chase → steffen.wilberg
Attachment #178088 -
Attachment is obsolete: true
Attachment #180354 -
Flags: review?(dbaron)
Attachment #180354 -
Flags: review?(dbaron) → review+
(That said, the copyright owner there is actually wrong too, since copyright is still generally held by contributors.)
| Reporter | ||
Comment 11•20 years ago
|
||
Comment on attachment 180354 [details] [diff] [review] match the about dialog: 1998 - 2005 Landed on the Aviary1.0.1 branch. Checking in browser/app/nsBrowserApp.cpp; /cvsroot/mozilla/browser/app/nsBrowserApp.cpp,v <-- nsBrowserApp.cpp new revision: 1.10.4.3.6.1; previous revision: 1.10.4.3 done
Attachment #180354 -
Flags: approval-aviary1.1a?
Attachment #180354 -
Flags: approval-aviary1.0.3+
| Reporter | ||
Comment 12•20 years ago
|
||
blocking-aviary1.0.3+, blocking-aviary1.0.4- added fixed-aviary1.0.3 keyword
Flags: blocking-aviary1.0.4?
Flags: blocking-aviary1.0.3?
Flags: blocking-aviary1.0.3+
Keywords: fixed-aviary1.0.3
Comment 13•20 years ago
|
||
Comment on attachment 180354 [details] [diff] [review] match the about dialog: 1998 - 2005 a=asa
Attachment #180354 -
Flags: approval-aviary1.1a? → approval-aviary1.1a+
| Reporter | ||
Comment 14•20 years ago
|
||
Comment on attachment 180354 [details] [diff] [review] match the about dialog: 1998 - 2005 Landed on the trunk. Checking in browser/app/nsBrowserApp.cpp; /cvsroot/mozilla/browser/app/nsBrowserApp.cpp,v <-- nsBrowserApp.cpp new revision: 1.24; previous revision: 1.23 done
| Reporter | ||
Comment 15•20 years ago
|
||
Resolving fixed. Please file any issues related to changing our established behaviour in what we print as a copyright message in a new bug.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Attachment #178088 -
Flags: approval-aviary1.0.5?
You need to log in
before you can comment on or make changes to this bug.
Description
•