Opened 17 years ago
Closed 14 years ago
#2191 closed enhancement (fixed)
Updated posix & bsd license headers
Reported by: | ekdahl | Owned by: | nobody |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | System | Version: | |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
Attached is a patch that updates the posix headers' licence headers according to the coding guidelines.
Attachments (3)
Change History (16)
by , 16 years ago
Attachment: | bsd_license_headers.diff added |
---|
comment:1 by , 16 years ago
Summary: | Updated posix license headers → Updated posix & bsd license headers |
---|
I've updated the patches to the current revision. Is there any interest in them or should I just drop them?
follow-up: 3 comment:2 by , 16 years ago
Thanks for your work! There is just a minor problem; while our coding style guide mentions "Public headers should always be copyrighted to "Haiku Inc." and don't list any authors.", this does not mean that you can deliberately change such headers. Especially "Copyright Marcus Overhagen" is not the same as "Copyright Haiku, Authors Marcus Overhagen".
Maybe you can just ask him if he is okay to be removed from that header (and such make it look like the others). If not, I would at least volunteer to rewrite the null.h header :-))
follow-up: 4 comment:3 by , 16 years ago
Replying to axeld:
Thanks for your work! There is just a minor problem; while our coding style guide mentions "Public headers should always be copyrighted to "Haiku Inc." and don't list any authors.", this does not mean that you can deliberately change such headers. Especially "Copyright Marcus Overhagen" is not the same as "Copyright Haiku, Authors Marcus Overhagen".
Maybe you can just ask him if he is okay to be removed from that header (and such make it look like the others). If not, I would at least volunteer to rewrite the null.h header :-))
Ok, so here's a new version with a rewritten null.h header then. But its hard to rewrite a such small header without getting a strong resemblance to the old one :)
BTW, some coding style questions struck me:
Whats the preferred header guard style, _NULL_H_, _NULL_H or NULL_H? I've seen all variants but no mention in the guidelines.
Should there be any indenting for preprocessor code?
#ifndef NULL #define NULL 0 #endif or #ifndef NULL # define NULL 0 #endif or #ifndef NULL
#define NULL 0
#endif (but with tabs)?
comment:4 by , 16 years ago
Replying to ekdahl:
Should there be any indenting for preprocessor code?
#ifndef NULL #define NULL 0 #endif or #ifndef NULL # define NULL 0 #endif or #ifndef NULL
#define NULL 0
#endif (but with tabs)?
Hmm, seems all my newlines disappeared on the trac page, but it looks alright in the email at least.
comment:5 by , 16 years ago
It wasn't just null.h, though. You can use \{\{\{/\}\}\} to have correct newlines :-) In any case, I usually do it this way, and that's also what you'll find in 95% our sources:
#ifndef NULL # define NULL 0 #endif
The header guards follow at least one rule: local/(very) private headers use NAME_H, public or shared system headers use at least _NAME_H (as the underscore is generally a reserved namespace for the operating system, not just Haiku). If it's _NAME_H_ or _NAME_H I don't really care, I would just follow what the majority of the other POSIX headers does.
comment:6 by , 16 years ago
Hi,
Axel is correct, you can't simply remove a copyright and replace it with a different one. Adding an author list will not heal this error.
Regarding the 3 posix headers with copyright attribution to me, I'll clear those up for sake of conformity myself (this afternoon).
But I still see a problem. The Copyright headers usually mention a year for example in wchar.t it's 2002. Looking at http://svn.berlios.de/viewcvs/haiku/haiku/trunk/headers/posix/wchar.h?rev=25433&view=log I see that this file has been worked on in 2002, 2003, 2005, 2006, 2007 and 2008.
Thus, instead of doing
- Copyright (C) 2002 Marcus Overhagen
+ Copyright 2002 Haiku Inc. All Rights Reserved.
I think I'll have to add + Copyright 2002-2003, 2005-2008 Haiku Inc. All Rights Reserved.
or
+ Copyright 2002, 2003, 2005-2008 Haiku Inc. All Rights Reserved.
what do you guys think?
comment:7 by , 16 years ago
I think while it's legally more correct, I also think that this can be pretty much neglected in practice. Having the first and the last year is at least the most interesting part of it, else we could probably also only mention the current year - earlier versions will carry the earlier copyright, and the document as is is clearly not copyrighted in a previous year.
So much for the logic, but I'm not a lawyer :-)
I would suggest we just keep the current practice of mentioning the first and the last year of changes.
comment:8 by , 16 years ago
I think mentioning current year (the day copyright holder(s) were altered) as first year in range would be legal-wise clearest case. This is from previous experience on similar case. As far as law goes copyright doesn't end even though years mentioned in the copyright statement don't include current year...
I partly agree in what axeld said about current practice being to include the last year of change, but I think better wording would be to change/add "end" year when file is modified next time (excluding changes to copyright statement), if and only if it already includes Haiku Inc. (or OpenBeOS) in copyright statement.
comment:9 by , 16 years ago
I've now added a new patch that doesn't touch the files which were copyrighted to Marcus. This is hopefully the last version :)
comment:10 by , 16 years ago
Actually, the licence name also should be fixed. I discussed this with Richard M Stallman at RMLL, and as he said, "MIT Licence" is misleading. See: http://www.gnu.org/licenses/license-list.html#X11License We should really check this, decide which we use, either X11 or BSD-revised, and use the correct name. Hmm that should likely be a separate ticket I suppose.
comment:11 by , 15 years ago
Owner: | changed from | to
---|---|
Version: | R1/pre-alpha1 |
comment:12 by , 15 years ago
patch: | 0 → 1 |
---|
comment:13 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Updated to hrev26976