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

Corinna Vinschen corinna-cygwin@cygwin.com
Thu Jul 21 14:09:51 GMT 2022


On Jul 21 09:50, Ken Brown wrote:
> 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.

Great, I pushed a patch to simplify the URLs and added a link from
the Cygwin homepage to the cygwin apps homepage.


Thanks,
Corinna


More information about the Cygwin-apps mailing list