Commit History vs User Convenience

Michael Dickens michaelld at macports.org
Mon Oct 8 14:28:43 UTC 2018


In my opinion: A single rev-bump is required when changes to the overall Portfile require it -- whether a single commit or multiple commits in a PR. The overall goal of a rev-bump is to force the port to be updated; it is a developer necessity & it's value is really not for the user's convenience.

I'd say so long as you note specifically in the commit message that a rev-bump -should- be done here but you're holding off for a future commit within the same PR, then that meets all of the basic requirements: it provides history & rationale, and still does the required rev-bump.

To be fair: One could, of course, clone the MP GIT repo & check out the interim commit (without the rev-bump) ... but, I don't see why anyone would want to do this & if they are sophisticated enough to do this then they can figure out the rev-bump issue too.

My US$0.02 worth ... - MLD

On Mon, Oct 8, 2018, at 10:13 AM, Marcus Calhoun-Lopez wrote:
> I was hoping to get some help with how best to balance commit history 
> and user convenience.
> 
> I would like to make two unrelated changed to the GCC ports.
> Each change requires a revision increase of all the GCC ports.
> 
> There seem to be a few options:
> 1) Create two separate pull requests and have them merged separately.
> 2) Create one pull request that increases the revision number only once 
> for the two unrelated changes. 
> 3) Create one pull request with several commits. Each commit increases 
> the revision number (for a total of two).
> 
> Personally, I believe option 3 is the best choice.
> The history remains clean, and nobody has to rebuild GCC twice in a 
> matter of a few days.
> I have created such a pull request 
> (https://github.com/macports/macports-ports/pull/2730).
> 
> However, the comments in the PR seem to indicate that option 2 or 3 is 
> preferable.
> 
> I was hoping to see if others had any strong feelings about this.
> Ultimately, it makes little difference to me, but we have had concerns 
> in the past about frequent rebuilds of large ports such as GCC.
> 
> Thanks,
> Marcus


More information about the macports-dev mailing list