Comments (9)
AFAIK migrating code (even porting to other programming language) does not grant copyright, similarly to what generally happens with book translations (original author keeps the copyright, translator does not acquire copyrights).
from maintainer-tools.
This is a good to excuse to discuss how licenses should be described in the py files.
@odony said that "explicit boilerplate should not be necessary - copyright law applies automatically".
Personally I like the idea of simplifying license boilerplate at the top of code files, but I think that you should still be explicit of what is the license and authors, because in case the file is taken apart from the module, you can still have that basic information.
from maintainer-tools.
I like the idea of large boiler plate in openerp.py + simple one on other files + add license logo.
Shall we set it as a guideline even if different from Odoo?
I tend to stick to Odoo's guidelines but we have already some other different rules (eg: PEP8) so I would be OK to diverge here
from maintainer-tools.
I'm seeing people totally removing copyright and license information from code files, replacing it with a "see the LICENSE file" or the "__openerp__.py
file".
IMO there are good reasons to not do so, but the fact is that there is nothing in the contributing guidelines preventing it.
For reference, this has also been discussed here: #91
from maintainer-tools.
I like the idea of large boiler plate in openerp.py + simple one on other files
IMHO that does not make much sense. Individual headers are for preventing that a file is picked to another project with a different license and loses the AGPL accidentally, which applies equally to __openerp__.py
and other .py
files.
Well, not equally... In fact, __openerp__.py
does not contain code at all. It could be an XML or JSON file. Yes, it ends with .py
, but it really is a data file.
from maintainer-tools.
Just spotted a new comer's PR adding a new (C) owner just for migration, which usually we don't do.
Still, I struggled to find official guidelines on this topic here https://github.com/OCA/maintainer-tools/blob/master/CONTRIBUTING.md
It's me, or we really miss few lines about this?
from maintainer-tools.
Yeah, there's no official statement about this. Sometimes it's difficult to define the border.
from maintainer-tools.
Well, can we simply state that if we don't change at least, say, 3 lines (or even 1?) of code we are not allowed to add your company/yourself to (C)?
Still, it's nice to add the guy to contribs.
from maintainer-tools.
IMO we could state that copyright comes with creative work.
You should get rewarded by adding yourself to the contibutors. Copyright needs at least adding features, bugfixes, rewriting the module.
@simahawk By any chance can you point us to some example?
from maintainer-tools.
Related Issues (20)
- pre-commit crashes in CI HOT 1
- Documentation for the best way to manage a high number of OCA repositories HOT 11
- I faced with error while using this module in pre-commit. HOT 1
- About squashing translation commits HOT 2
- Update our tooling to support Odoo 17 HOT 20
- Mention `odoo-module-migrator` in migration procedure wiki page HOT 1
- Mention `oca-port` in migration procedure wiki page HOT 6
- oca-gen-addon-readme: exits with no error if `description.rst` is not present HOT 14
- Update migration v17 Wiki HOT 2
- Clarify CONTRIBUTOR.md formatting HOT 20
- Wiki migration 17.0: New Settings format HOT 1
- oca-gen-addon-readme: stop emitting an XML declaration HOT 1
- Fail to run pre-commit on OCA project: ERROR No matching distribution found for setuptools_scm HOT 2
- Update wiki "Migration to version 16.0" with `fields_view_get` becomes `get_view` HOT 2
- Mention data-hotkey in the 15.0 migration guide
- delete
- WIKI for documentation HOT 6
- module 'github3' has no attribute 'authorize'
- copy_maintainers script fails syncing the big teams HOT 7
- [17.0][mig] partner_statement: Migration to 17.0 HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from maintainer-tools.