<div dir="ltr"><br><br>On Fri, Jul 12, 2019 at 4:23 PM Huidae Cho <<a href="mailto:grass4u@gmail.com">grass4u@gmail.com</a>> wrote:<br>><br>> Greetings Developers,<br>><br>> In <a href="https://trac.osgeo.org/grass/wiki/HowToGit">https://trac.osgeo.org/grass/wiki/HowToGit</a>, it is not clear who is responsible for merging PRs when. Are individual core developers encouraged (allowed?) or discouraged to merge them? I'm talking about core, not addons. How does it work?<br>><br>> The main reason why I ask this question is because merging PRs into the upstream master takes time and I cannot wait until that happens (who, when?) to compile all my PRs into one build.<br>><br>> If you create multiple branches off of master and create PRs, it wouldn't be easy to keep track of all those individual branches in one place and compile the personal "latest" version until the PRs get merged. For example,<br>><br>> 1. master => feature1<br>> 2. master => feature2<br>><br>> Compiling feature1 and feature2 branches would only give me feature1 and feature2, respectively, but not both. Should I do something like this?<br>><br>> 1. master => feature1<br>> 2. feature1 => feature2<br>><br>> Then, how can we make sure that there won't be any conflicts between my feature2 (<= feature1) branch and the OSGeo master?<br>><br>> Not sure if what I'm doing is "right," but I'm doing this:<br>><br>> 1. create a personal master branch "hcho" (yes, my name)<br>> 2. master => feature1<br>> 3. master => feature2<br>> 4. merge feature1 and feature2 into "hcho"<br>> 5. compile "hcho"<br>><br>> To be honest, it's a lot of work :-(. Any suggestions or hints would be very welcome.<br><div><br></div><div>Yes, I agree, this is a lot of work. git is convenient for contributors, but a lot of work (much more than with svn) for maintainers.</div><div><br></div><div>If your PRs are incremental, i.e PR for feature 2 requires PR for feature 1, it would be easier to have only one PR with several commits.</div><div><br></div><div>I tend to merge your PRs related to MS Windows builds, but I can not test. OTOH, testing by others occurs more likely if the changes are in master, not in a PR.</div><div><br></div><div>Any opinions on merging PRs related to MS Windows builds to master without testing by another person?<br></div><div><br></div><div>Markus M<br></div><br></div>