[PATCH] Misc: win32 sources now preserved in release tarballs

[Maxim Dounin]

Hello!

On Mon, Jun 20, 2022 at 10:10:39AM +0400, Sergey Kandaurov wrote:

> > On 18 Jun 2022, at 00:37, Maxim Dounin <mdounin at mdounin.ru> wrote:
> > 
> > On Fri, Jun 17, 2022 at 07:31:27PM +0400, Sergey Kandaurov wrote:
> > 
> >>> On 17 Jun 2022, at 01:33, Maxim Dounin <mdounin at mdounin.ru> wrote:
> >>> 
> >>> # HG changeset patch
> >>> # User Maxim Dounin <mdounin at mdounin.ru>
> >>> # Date 1655415152 -10800
> >>> # Fri Jun 17 00:32:32 2022 +0300
> >>> # Node ID e23a385cd0ec866a3eb1d8c9c956991e1ed50d78
> >>> # Parent 861f076eab72dab85bd5eaa56da770f8b10aa8ad
> >>> Misc: win32 sources now preserved in release tarballs.
> >>> 
> >>> This makes it possible to build nginx under Windows from release tarballs
> >>> instead of using source code repository.
> >>> 
> >>> diff --git a/misc/GNUmakefile b/misc/GNUmakefile
> >>> --- a/misc/GNUmakefile
> >>> +++ b/misc/GNUmakefile
> >>> @@ -15,12 +15,6 @@ release: export
> >>> 
> >>> 	mv $(TEMP)/$(NGINX)/auto/configure $(TEMP)/$(NGINX)
> >>> 
> >> 
> >> Do we need the empty line here?
> >> It could be collapsed if removing all the below.
> > 
> > It clearly separates two distinct blocks: configure and docs. I 
> > don't think this distinction have to be removed.
> > 
> 
> Ok.
> 
> >>> -	# delete incomplete sources
> >>> -	rm $(TEMP)/$(NGINX)/src/event/ngx_event_acceptex.c
> >>> -	rm $(TEMP)/$(NGINX)/src/event/ngx_event_connectex.c
> >> 
> >> I wonder if we need to touch ngx_event_connectex.c.
> >> It's not used nor connected to the build since introduction,
> >> so it does look incomplete.
> > 
> > I don't think it is a good idea to keep any of these removals, 
> > even if some files are not connected to the build. Rather, I 
> > think that release tarballs should be as close as possible to the 
> > source code repository.
> 
> Agreed, it makes sense.

Pushed to http://mdounin.ru/hg/nginx/.

-- 
Maxim Dounin
http://mdounin.ru/