[PATCH rebase] Make rebaseall a wrapper around the autorebase postinstall, script

Ken Brown kbrown@cornell.edu
Thu Jul 21 13:50:40 GMT 2022


On 7/21/2022 5:09 AM, Corinna Vinschen wrote:
> We're using the "?p=" URLs on the cygin-apps website, too.  Maybe it's
> just me, but to me it looks better if we change this to the 2nd form
> where possible.  What do you think?

I agree that it looks better without the ?p=.  I have no idea why the latter is 
used on the cygwin-apps website.

> Just a style issue, but the two functions should stay first in the file.
> Please move this code just right before the trap call.
> 
> Other than that, LGTM.

I've made all the changes you suggested and pushed it.

Ken


More information about the Cygwin-apps mailing list