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
•