Re: [Mingw-w64-public] [PATCH 2/2] widl: Quote path names in the makefile, to fix installing into a path that contains spaces

2019-12-02 Thread Martin Storsjö

On Sun, 1 Dec 2019, JonY via Mingw-w64-public wrote:


On 11/30/19 8:28 PM, Martin Storsjö wrote:

Signed-off-by: Martin Storsjö 
---
 mingw-w64-tools/widl/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mingw-w64-tools/widl/Makefile.am b/mingw-w64-tools/widl/Makefile.am
index 0cc45611e..7c570210e 100644
--- a/mingw-w64-tools/widl/Makefile.am
+++ b/mingw-w64-tools/widl/Makefile.am
@@ -58,7 +58,7 @@ widl_SOURCES = src/widl.h \
   include/pathtools.h \
   include/winnt.rh

-widl_CPPFLAGS = -I$(top_srcdir)/include -DDEFAULT_INCLUDE_DIR=\"@WIDL_INCLUDEDIR@\" 
-DDEFAULT_BINDIR=\"$(bindir)\"
+widl_CPPFLAGS = -I$(top_srcdir)/include -DDEFAULT_INCLUDE_DIR=\""@WIDL_INCLUDEDIR@"\" 
-DDEFAULT_BINDIR=\""$(bindir)"\"
 widl_CFLAGS = -O3 -g -Wall -Wformat -Wpacked -Wmissing-declarations 
-Wimplicit-function-declaration -Wmissing-prototypes -Wstrict-aliasing=2

 DISTCHECK_CONFIGURE_FLAGS = --host=$(host) --target=$(target)



Patches look good to me.


Thanks, pushed them both.

// Martin

___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public


Re: [Mingw-w64-public] [PATCH 2/2] widl: Quote path names in the makefile, to fix installing into a path that contains spaces

2019-11-30 Thread JonY via Mingw-w64-public
On 11/30/19 8:28 PM, Martin Storsjö wrote:
> Signed-off-by: Martin Storsjö 
> ---
>  mingw-w64-tools/widl/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/mingw-w64-tools/widl/Makefile.am 
> b/mingw-w64-tools/widl/Makefile.am
> index 0cc45611e..7c570210e 100644
> --- a/mingw-w64-tools/widl/Makefile.am
> +++ b/mingw-w64-tools/widl/Makefile.am
> @@ -58,7 +58,7 @@ widl_SOURCES = src/widl.h \
>include/pathtools.h \
>include/winnt.rh
>  
> -widl_CPPFLAGS = -I$(top_srcdir)/include 
> -DDEFAULT_INCLUDE_DIR=\"@WIDL_INCLUDEDIR@\" -DDEFAULT_BINDIR=\"$(bindir)\"
> +widl_CPPFLAGS = -I$(top_srcdir)/include 
> -DDEFAULT_INCLUDE_DIR=\""@WIDL_INCLUDEDIR@"\" -DDEFAULT_BINDIR=\""$(bindir)"\"
>  widl_CFLAGS = -O3 -g -Wall -Wformat -Wpacked -Wmissing-declarations 
> -Wimplicit-function-declaration -Wmissing-prototypes -Wstrict-aliasing=2
>  
>  DISTCHECK_CONFIGURE_FLAGS = --host=$(host) --target=$(target)
> 

Patches look good to me.


signature.asc
Description: OpenPGP digital signature
___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public


[Mingw-w64-public] [PATCH 2/2] widl: Quote path names in the makefile, to fix installing into a path that contains spaces

2019-11-30 Thread Martin Storsjö
Signed-off-by: Martin Storsjö 
---
 mingw-w64-tools/widl/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mingw-w64-tools/widl/Makefile.am b/mingw-w64-tools/widl/Makefile.am
index 0cc45611e..7c570210e 100644
--- a/mingw-w64-tools/widl/Makefile.am
+++ b/mingw-w64-tools/widl/Makefile.am
@@ -58,7 +58,7 @@ widl_SOURCES = src/widl.h \
   include/pathtools.h \
   include/winnt.rh
 
-widl_CPPFLAGS = -I$(top_srcdir)/include 
-DDEFAULT_INCLUDE_DIR=\"@WIDL_INCLUDEDIR@\" -DDEFAULT_BINDIR=\"$(bindir)\"
+widl_CPPFLAGS = -I$(top_srcdir)/include 
-DDEFAULT_INCLUDE_DIR=\""@WIDL_INCLUDEDIR@"\" -DDEFAULT_BINDIR=\""$(bindir)"\"
 widl_CFLAGS = -O3 -g -Wall -Wformat -Wpacked -Wmissing-declarations 
-Wimplicit-function-declaration -Wmissing-prototypes -Wstrict-aliasing=2
 
 DISTCHECK_CONFIGURE_FLAGS = --host=$(host) --target=$(target)
-- 
2.17.1



___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public