Bug 7263 - Mail::SpamAssassin module is missing the copyright
Summary: Mail::SpamAssassin module is missing the copyright
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Building & Packaging (show other bugs)
Version: 3.4.1
Hardware: PC Windows 7
: P2 normal
Target Milestone: 3.4.2
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-11 02:18 UTC by Quanah Gibson-Mount
Modified: 2017-04-15 09:57 UTC (History)
3 users (show)



Attachment Type Modified Status Actions Submitter/CLA Status

Note You need to log in before you can comment on or make changes to this bug.
Description Quanah Gibson-Mount 2015-11-11 02:18:51 UTC
While there is a Copyright section in the POD documentation for Mail::SpamAssassin, it lists no copyright at all.  Instead, it refers to the license, which isn't quite the same thing.

I'd expect something like:

Copyright 2001-2015 the Apache Foundation
Comment 1 Kevin A. McGrail 2015-11-11 17:25:24 UTC
A good thing to polish since.

I think these are the applicable instructions from the license to apply to the code and the first line with the copyright is missing.

Copyright [yyyy] [name of copyright owner]

   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
   You may obtain a copy of the License at

       http://www.apache.org/licenses/LICENSE-2.0

   Unless required by applicable law or agreed to in writing, software
   distributed under the License is distributed on an "AS IS" BASIS,
   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   See the License for the specific language governing permissions and
   limitations under the License.


Suggest we add use the Copyright (C) 2015 The Apache Software Foundation to any file with @LICENSE.

And the release process should be updated to make sure the copyright is updated.

Thoughts?
Comment 2 Quanah Gibson-Mount 2015-11-11 17:39:51 UTC
Copyright usually starts from the starting point of the package.  I noted 2001 specifically, as that was the oldest year I could find a SpamAssassin release for.  I assume it has been a part of the apache foundation that entire time.  If not, there may need to be more than one copyright listed.
Comment 3 Kevin A. McGrail 2015-11-11 17:43:44 UTC
The project was brought into the ASF and my thought is baby steps because I know what is correct now.  Figuring out the other information adds no value and could be a large rabbithole.  But if someone knows the correct, factual information, my thought is to add it.  Absent that information, this is a triage step for a good point.
Comment 4 Quanah Gibson-Mount 2015-11-11 17:53:43 UTC
Ok.  Well, we know it's been in the ASF I assume since at least X date, so could start that copyright then? like 2010-2015 ASF etc?
Comment 5 Kevin A. McGrail 2015-11-11 18:23:35 UTC
I figure to start with the information I can defend.
Comment 6 Quanah Gibson-Mount 2015-11-11 18:39:16 UTC
SA joined the ASF in 2004:

https://en.wikipedia.org/wiki/SpamAssassin#History

We can see the move to do the work in 2003:

http://sourceforge.net/p/spamassassin/mailman/message/16868996/
Comment 7 Quanah Gibson-Mount 2015-11-11 21:30:57 UTC
From backpan, we can see that SA 3.0.1, released in 22 Oct 2004, was definitely part of the ASF:

http://backpan.perl.org/authors/id/J/JM/JMASON/Mail-SpamAssassin-3.0.1.readme

So it was definitely part of ASF for that release.

The release just prior to that (2.64) from 04 Aug 2004, was at the old spamassassin.sourceforge.net website, and does not reference ASF.

So 2004 is the correct starting year for SA as part of ASF. :)

--Quanah
Comment 8 Quanah Gibson-Mount 2015-11-11 21:31:19 UTC
2.64 readme, for reference:

http://backpan.perl.org/authors/id/J/JM/JMASON/Mail-SpamAssassin-2.64.readme
Comment 9 Kevin A. McGrail 2015-11-16 14:24:22 UTC
For now, I just used the current copyright year and updated the build doc to review the Credits for accurate Copyright.

3.4:
Sending        CREDITS
Sending        build/README
Sending        sa-compile.raw
Sending        sa-update.raw
Sending        spamassassin.raw
Transmitting file data .....
Committed revision 1714589.

trunk:
Trunk is same for 1714591 + a spamdcheck under revision 1714592.
Comment 10 Sidney Markowitz 2017-04-14 23:26:35 UTC
Committed to 3.4, revision 1791428 sa-check_spamd.raw port of revision 1714592 which was also part of this issue.
Comment 11 Sidney Markowitz 2017-04-14 23:41:54 UTC
Committed revision 1791430
The change to spamassassin.raw was committed to 3.4 branch but had never been committed to trunk.