Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0002: Upstreaming #2

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

marcusmueller
Copy link
Member

No description provided.

@mbr0wn
Copy link
Member

mbr0wn commented Feb 23, 2017

We're not yet accepting PRs on this repo.

@noc0lour
Copy link
Member

noc0lour commented Mar 3, 2018

Is this PR actually up for discussion right now?

@marcusmueller
Copy link
Member Author

@noc0lour Yes, discussion is definitely open.

@mbr0wn I'll have to fix this template, it does lose its number as per our process, right?

@mbr0wn
Copy link
Member

mbr0wn commented Mar 4, 2018

As per GREP0, you and I pick a number when we merge. That said, we said we wanted to reserve a number space for (0-9) for "special" GREPs, and this might qualify.

@mbr0wn
Copy link
Member

mbr0wn commented Mar 4, 2018

This one probably needs some rework since it predates the inauguration of the offical GREP process by a year.

@marcusmueller
Copy link
Member Author

Yes, being ahead of time is what GNU Radio Maintainers do 😉

I'm going to rework this tomorrow noon.

This not only includes replacing all occurences of GRIPE by GREP, but
also adopting the GREP process as preferred way of bringing in
substantial change to the environment.
@marcusmueller
Copy link
Member Author

Reworked.


## Copyright / License

[WTFPL](http://www.wtfpl.net/txt/copying/) Version 2 or later
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Every open source lawyer will tell you this is not a license. Please choose something more approriate.

Copy link
Member Author

@marcusmueller marcusmueller Mar 5, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://www.gnu.org/licenses/license-list.html#WTFPL it's not recommended by the FSF, but recognized. I frankly think that the spirit of the WTFPL fully applies to this document, and thus I'd very much like to keep that license.

@mbr0wn
Copy link
Member

mbr0wn commented Mar 5, 2018

The second half of this document are generic contribution instructions. @noc0lour has also brought up such a document, plus, there's an outdated wiki page on contributing (https://wiki.gnuradio.org/index.php/Development). How about this: We make the sure the contribution info is up to date (maybe turn it into a GREP, too). That should include all the mechanics (forking, CLAs...). Then we can focus on a document that says "when do I upstream my block vs keeping it in my OOT".

For the latter case, this document lacks some details on how it stays maintainable once upstreamed. Unit tests are nice, but we should also work on ways to have higher-level tests. That remains TBD.

@marcusmueller
Copy link
Member Author

not quite sure what document of @noc0lour you're referring to here, but I'd very much like to read it in that case 👍

I think the part about "when to upstream and when to keep in OOT" is indeed expandable, but I do see a clear checklist – I intended this GREP to be exactly the document that defines that, so I'm happy to see that you agree we need something like that. I'm not quite sure what to make of your criticism here:

  • in which way specifically is the document lacking a guideline when to upstream and when not to?
  • how would you approach describing the process of "keeping things maintainable once upstreamed"? To me it's writing tests at the time of upstreaming, and keeping those tests running when evolving the project. Is there something higher-level to it that I'm missing (honest question from a newbie maintainer)?

@noc0lour
Copy link
Member

noc0lour commented Mar 5, 2018

I was just referring in some chats that I'd very much like to write a technical "Contributors"-Guide once we are done with formatting and dev-model changing. I'm in for writing a the technical part explaining:

  • how should your code/contribution look like (abstract/high-level)
  • which tools to use
  • timeline/steps to take (e.g. CLA signing)
  • where to ask for help

This should help kickstart new contributors and reduce the load of questions we already have an clear, but hard to find, answer.

Re tests: We should try separating unit tests, integration tests (how well to certain modules play nice together?) and system tests (Is a full transceiver still working [with HW] ?).

Copy link
Member

@noc0lour noc0lour left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small typos, general idea of this GREP is good though.


GNU Radio calls itself the "Free and Open Software Radio Ecosystem". We're not
doing that bad with ecosystem size, but the code quality in Out-Of-Tree Modules
(OOTs) is very varying, and with every release, we risk serious bitrot, since
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very

### Upstreaming Process

1. The party interested in upstreaming MUST integrate their changes into a fork
of GNU Radio on a publicly available git repository (classically: Github),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitHub

@noc0lour
Copy link
Member

noc0lour commented Mar 6, 2018

I'd rather not specify the technical side of "How to contribute" in a GREP but in a Contributor's Guide (e.g. in the wiki) and reference it from README.md.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants