[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