Closed Bug 286957 Opened 20 years ago Closed 20 years ago

bump copyright year in browser/app/nsBrowserApp.cpp

Categories

(Firefox :: General, defect)

1.0 Branch
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: chase, Assigned: steffen.wilberg)

Details

(Keywords: fixed-aviary1.0.3)

Attachments

(1 file, 1 obsolete file)

The copyright year needs to be bumped in browser/app/nsBrowserApp.cpp.
Nominating blocking-aviary1.0.3 and blocking-aviary1.1.
Status: NEW → ASSIGNED
Flags: blocking-aviary1.1?
Flags: blocking-aviary1.0.3?
Hardware: PC → All
Attached patch patch (obsolete) — Splinter Review
patch
Attachment #178088 - Flags: review?
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+
> 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? :-)
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 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-
Flags: blocking-aviary1.0.4?
Flags: blocking-aviary1.1? → blocking-aviary1.1+
Assignee: chase → steffen.wilberg
Attachment #178088 - Attachment is obsolete: true
Attachment #180354 - Flags: review?(dbaron)
(That said, the copyright owner there is actually wrong too, since copyright is still generally held by contributors.)
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+
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+
Comment on attachment 180354 [details] [diff] [review] match the about dialog: 1998 - 2005 a=asa
Attachment #180354 - Flags: approval-aviary1.1a? → approval-aviary1.1a+
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
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
vrfy'd fixed in trunk and branch via lxr.
Status: RESOLVED → VERIFIED
Attachment #178088 - Flags: approval-aviary1.0.5?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: