<div dir="ltr"><div>The main advantage of having all grass-addons in one repo is that they are easy to manage (for core devs).</div><div><br></div><div>The main disadvantage of having all grass-addons in one repo is that a contributor concerned about his single addon needs to get the whole repo, wasting disk space.</div><div><br></div><div>There are probably two different scenarios regarding contributions to grass-addons:</div><div>1. a new addon: needs IMHO a PR, also if the contributor has write access to the addons repo<br></div><div>2. modifications to an existing addon (bug fix, updated manual, new functionality, etc): if the contributor has properly validated the changes, the modifications could be pushed directly to the repo, without a PR. But since even people (more or less) familiar with git have different opinions on how to use git the correct way (as many correct ways as people having an opinion on it), it is probably safer to go through a PR.</div><div><br></div><div>my0.2cents,</div><div><br></div><div>Markus M  <br></div><div><br>On Fri, May 24, 2019 at 4:25 PM Stefan Blumentrath <<a href="mailto:Stefan.Blumentrath@nina.no">Stefan.Blumentrath@nina.no</a>> wrote:<br>><br>> Hi,<br>><br>> Collecting addons in a central repo seems very valuable to me too, for all the reasons Vacslav mentioned.<br>><br>> I am no git expert either, but PRs should not be a big issue to do (unless you are VERY productive). People could merge their own PRs, no? Creating a PR, does not mean it has to be reviewed by another dev, right? In my organization colleagues even usr PRs for repos where they are the only contributor...<br>><br>> I would argue having procedures as equal as possible between addons and core is just beneficial. Less confusion, fewer guidelines to maimtain, possibility to have CI before things are merged, and training for new devs that evolve from addon-dev to core-dev...<br>><br>> Cheers<br>> Stefan<br>><br>><br>> From: Anna Petrášová<br>> Sent: Friday 24 May 15:57<br>> Subject: Re: [GRASS-dev] grass-addons on github<br>> To: Martin Landa<br>> Cc: Paulo van Breugel, GRASS developers email list<br>><br>><br>><br>><br>> On Fri, May 24, 2019 at 3:55 AM Martin Landa <<a href="mailto:landa.martin@gmail.com">landa.martin@gmail.com</a>> wrote:<br>><br>> Hi,<br>><br>> pá 24. 5. 2019 v 8:48 odesílatel Paulo van Breugel<br>> <<a href="mailto:p.vanbreugel@gmail.com">p.vanbreugel@gmail.com</a>> napsal:<br>> > I have read about the procedure for contributors to the main grass repository. Question is, how are we going to deal with add-ons?<br>> ><br>> > Are we working with a central repository (OSGeo/grass-addons) and follow the same protocol as for OSGEO/grass. If so, who will be responsible for approving pull requests? An alternative more like the old situation is that authors will be able to directly commit to the addon repository.<br>><br>> in my opinion requesting PRs for `grass-addons` is maybe overkill. It<br>><br>><br>><br>> If we don't care about the history and any mess in the grass-addons repository, then yes, we don't need pull requests.<br>> But a lot of people who might be contributing there might not be familiar with the peculiarities of git (since even most core grass devs including me aren't), so eventually we will end up with a lot of mess, which somebody will need to clean up. PR is a standard way to work on GitHub, so let's use it. The same approach as for the main grass repo could be used.<br>>  <br>><br>> must be somehow discussed anyway. If we suggest direct commits it's<br>> important to avoid not needed 'merge from master' commits [1]. The<br>> workflow must be clear (rebase always) to avoid such situations. It<br>><br>><br>> I don't quite get how to use rebase yet, but that's the issue, it seems that if you use it incorrectly, it can be dangerous.<br>>  <br>><br>> was not defined yet. Even suggested workflow related to the main<br>> repository is not clearly defined [2]. This must be improved in a near<br>> future.<br>><br>> > Or should add-on authors maintain their own repositories, and will there be a way to provide links to the authors repositories in a central place?<br>><br>><br>> We did this with couple more complicated addons, we do internal development in our git and then push it to the main repo when we want. I like the idea of having all addons in one repository, then you can provide the Windows binaries for them, that is also an incentive for contributers to put it there (you get windows binary, hosting of manuals, simple installation). But I get people want the distributed approach too.<br>><br>> Anna<br>><br>><br>> Would be nice if g.extension (wingrass builds) supports distributed<br>> personal repos. I can imagine that it could be driven by a metadata<br>> file stored in central `grass-addons` repo. But someone need to<br>> implement it (g.extension, manual pages builds and wingrass builds).<br>> Would be cool.<br>><br>> > With a central repository for all add-ons I guess it will be easier to maintain an overview like <a href="https://grass.osgeo.org/grass76/manuals/addons/">https://grass.osgeo.org/grass76/manuals/addons/</a> and to create the windows binaries?<br>><br>> Sure. But see my notes above.<br>><br>> Ma<br>><br>> [1] <a href="https://lists.osgeo.org/pipermail/grass-dev/2019-May/092663.html">https://lists.osgeo.org/pipermail/grass-dev/2019-May/092663.html</a><br>> [2] <a href="https://trac.osgeo.org/grass/wiki/HowToGit">https://trac.osgeo.org/grass/wiki/HowToGit</a><br>><br>> --<br>> Martin Landa<br>> <a href="http://geo.fsv.cvut.cz/gwiki/Landa">http://geo.fsv.cvut.cz/gwiki/Landa</a><br>> <a href="http://gismentors.cz/mentors/landa">http://gismentors.cz/mentors/landa</a><br>> _______________________________________________<br>> grass-dev mailing list<br>> <a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>> <a href="https://lists.osgeo.org/mailman/listinfo/grass-dev">https://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>><br>><br>><br>> _______________________________________________<br>> grass-dev mailing list<br>> <a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>> <a href="https://lists.osgeo.org/mailman/listinfo/grass-dev">https://lists.osgeo.org/mailman/listinfo/grass-dev</a></div></div>