public inbox for newlib@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] libgloss: riscv: Install machine/syscall.h
@ 2022-10-18 19:53 Simon Cook
  2022-10-19 10:47 ` Corinna Vinschen
  0 siblings, 1 reply; 6+ messages in thread
From: Simon Cook @ 2022-10-18 19:53 UTC (permalink / raw)
  To: newlib

A recent change to the Makefile.in for riscv resulted in the
machine/syscall.h header not being installed. This updates the file
to install this file again.

Signed-off-by: Simon Cook <simon.cook@embecosm.com>
---
 libgloss/Makefile.in        | 68 +++++++++++++++++++++++++------------
 libgloss/riscv/Makefile.inc |  3 ++
 2 files changed, 50 insertions(+), 21 deletions(-)

diff --git a/libgloss/Makefile.in b/libgloss/Makefile.in
index 16723dca1..36ab3fbd3 100644
--- a/libgloss/Makefile.in
+++ b/libgloss/Makefile.in
@@ -212,8 +212,9 @@ am__uninstall_files_from_dir = { \
          $(am__cd) "$$dir" && rm -f $$files; }; \
   }
 am__installdirs = "$(DESTDIR)$(multilibtooldir)" "$(DESTDIR)$(bindir)" \
- "$(DESTDIR)$(infodir)" "$(DESTDIR)$(includesystooldir)" \
- "$(DESTDIR)$(includetooldir)" "$(DESTDIR)$(multilibtooldir)"
+ "$(DESTDIR)$(infodir)" "$(DESTDIR)$(includemachinetooldir)" \
+ "$(DESTDIR)$(includesystooldir)" "$(DESTDIR)$(includetooldir)" \
+ "$(DESTDIR)$(multilibtooldir)"
 LIBRARIES = $(multilibtool_LIBRARIES) $(noinst_LIBRARIES)
 ARFLAGS = cru
 AM_V_AR = $(am__v_AR_@AM_V@)
@@ -471,7 +472,8 @@ am__can_run_installinfo = \
     n|no|NO) false;; \
     *) (install-info --version) >/dev/null 2>&1;; \
   esac
-DATA = $(includesystool_DATA) $(includetool_DATA) $(multilibtool_DATA)
+DATA = $(includemachinetool_DATA) $(includesystool_DATA) \
+ $(includetool_DATA) $(multilibtool_DATA)
 RECURSIVE_CLEAN_TARGETS = mostlyclean-recursive clean-recursive \
   distclean-recursive maintainer-clean-recursive
 am__recursive_targets = \
@@ -610,7 +612,6 @@ pdfdir = @pdfdir@
 prefix = @prefix@
 program_transform_name = @program_transform_name@
 psdir = @psdir@
-runstatedir = @runstatedir@
 sbindir = @sbindir@
 sharedstatedir = @sharedstatedir@
 srcdir = @srcdir@
@@ -791,6 +792,8 @@ TEXINFO_TEX = ../texinfo/texinfo.tex
 @CONFIG_RISCV_TRUE@ riscv/semihost-sys_unlink.c \
 @CONFIG_RISCV_TRUE@ riscv/semihost-sys_write.c

+@CONFIG_RISCV_TRUE@includemachinetooldir = $(tooldir)/include/machine
+@CONFIG_RISCV_TRUE@includemachinetool_DATA = riscv/machine/syscall.h
 @CONFIG_WINCE_TRUE@gdbdir = ${dir ${patsubst %/,%,${dir @srcdir@}}}gdb
 @CONFIG_WINCE_TRUE@wince_stub_exe_SOURCES = wince-stub.c
 @CONFIG_WINCE_TRUE@wince_stub_exe_CPPFLAGS = $(AM_CPPFLAGS) -I$(gdbdir)
@@ -2504,6 +2507,27 @@ maintainer-clean-aminfo:
   echo " rm -f $$i $$i-[0-9] $$i-[0-9][0-9] $$i_i[0-9] $$i_i[0-9][0-9]"; \
   rm -f $$i $$i-[0-9] $$i-[0-9][0-9] $$i_i[0-9] $$i_i[0-9][0-9]; \
  done
+install-includemachinetoolDATA: $(includemachinetool_DATA)
+ @$(NORMAL_INSTALL)
+ @list='$(includemachinetool_DATA)'; test -n
"$(includemachinetooldir)" || list=; \
+ if test -n "$$list"; then \
+  echo " $(MKDIR_P) '$(DESTDIR)$(includemachinetooldir)'"; \
+  $(MKDIR_P) "$(DESTDIR)$(includemachinetooldir)" || exit 1; \
+ fi; \
+ for p in $$list; do \
+  if test -f "$$p"; then d=; else d="$(srcdir)/"; fi; \
+  echo "$$d$$p"; \
+ done | $(am__base_list) | \
+ while read files; do \
+  echo " $(INSTALL_DATA) $$files '$(DESTDIR)$(includemachinetooldir)'"; \
+  $(INSTALL_DATA) $$files "$(DESTDIR)$(includemachinetooldir)" || exit $$?; \
+ done
+
+uninstall-includemachinetoolDATA:
+ @$(NORMAL_UNINSTALL)
+ @list='$(includemachinetool_DATA)'; test -n
"$(includemachinetooldir)" || list=; \
+ files=`for p in $$list; do echo $$p; done | sed -e 's|^.*/||'`; \
+ dir='$(DESTDIR)$(includemachinetooldir)'; $(am__uninstall_files_from_dir)
 install-includesystoolDATA: $(includesystool_DATA)
  @$(NORMAL_INSTALL)
  @list='$(includesystool_DATA)'; test -n "$(includesystooldir)" || list=; \
@@ -2680,7 +2704,7 @@ all-am: Makefile $(INFO_DEPS) $(LIBRARIES)
$(PROGRAMS) $(DATA) \
  config.h all-local
 installdirs: installdirs-recursive
 installdirs-am:
- for dir in "$(DESTDIR)$(multilibtooldir)" "$(DESTDIR)$(bindir)"
"$(DESTDIR)$(infodir)" "$(DESTDIR)$(includesystooldir)"
"$(DESTDIR)$(includetooldir)" "$(DESTDIR)$(multilibtooldir)"; do \
+ for dir in "$(DESTDIR)$(multilibtooldir)" "$(DESTDIR)$(bindir)"
"$(DESTDIR)$(infodir)" "$(DESTDIR)$(includemachinetooldir)"
"$(DESTDIR)$(includesystooldir)" "$(DESTDIR)$(includetooldir)"
"$(DESTDIR)$(multilibtooldir)"; do \
   test -z "$$dir" || $(MKDIR_P) "$$dir"; \
  done
 install: install-recursive
@@ -2750,7 +2774,8 @@ info: info-recursive

 info-am: $(INFO_DEPS)

-install-data-am: install-includesystoolDATA install-includetoolDATA \
+install-data-am: install-includemachinetoolDATA \
+ install-includesystoolDATA install-includetoolDATA \
  install-info-am install-multilibtoolDATA \
  install-multilibtoolLIBRARIES

@@ -2893,10 +2918,10 @@ ps: ps-recursive
 ps-am: $(PSS)

 uninstall-am: uninstall-binPROGRAMS uninstall-dvi-am uninstall-html-am \
- uninstall-includesystoolDATA uninstall-includetoolDATA \
- uninstall-info-am uninstall-multilibtoolDATA \
- uninstall-multilibtoolLIBRARIES uninstall-pdf-am \
- uninstall-ps-am
+ uninstall-includemachinetoolDATA uninstall-includesystoolDATA \
+ uninstall-includetoolDATA uninstall-info-am \
+ uninstall-multilibtoolDATA uninstall-multilibtoolLIBRARIES \
+ uninstall-pdf-am uninstall-ps-am

 .MAKE: $(am__recursive_targets) all check-am install-am install-strip

@@ -2910,17 +2935,18 @@ uninstall-am: uninstall-binPROGRAMS
uninstall-dvi-am uninstall-html-am \
  html-am info info-am install install-am install-binPROGRAMS \
  install-data install-data-am install-dvi install-dvi-am \
  install-exec install-exec-am install-exec-local install-html \
- install-html-am install-includesystoolDATA \
- install-includetoolDATA install-info install-info-am \
- install-man install-multilibtoolDATA \
- install-multilibtoolLIBRARIES install-pdf install-pdf-am \
- install-ps install-ps-am install-strip installcheck \
- installcheck-am installdirs installdirs-am maintainer-clean \
- maintainer-clean-aminfo maintainer-clean-generic \
- maintainer-clean-local mostlyclean mostlyclean-aminfo \
- mostlyclean-compile mostlyclean-generic mostlyclean-local pdf \
- pdf-am ps ps-am tags tags-am uninstall uninstall-am \
- uninstall-binPROGRAMS uninstall-dvi-am uninstall-html-am \
+ install-html-am install-includemachinetoolDATA \
+ install-includesystoolDATA install-includetoolDATA \
+ install-info install-info-am install-man \
+ install-multilibtoolDATA install-multilibtoolLIBRARIES \
+ install-pdf install-pdf-am install-ps install-ps-am \
+ install-strip installcheck installcheck-am installdirs \
+ installdirs-am maintainer-clean maintainer-clean-aminfo \
+ maintainer-clean-generic maintainer-clean-local mostlyclean \
+ mostlyclean-aminfo mostlyclean-compile mostlyclean-generic \
+ mostlyclean-local pdf pdf-am ps ps-am tags tags-am uninstall \
+ uninstall-am uninstall-binPROGRAMS uninstall-dvi-am \
+ uninstall-html-am uninstall-includemachinetoolDATA \
  uninstall-includesystoolDATA uninstall-includetoolDATA \
  uninstall-info-am uninstall-multilibtoolDATA \
  uninstall-multilibtoolLIBRARIES uninstall-pdf-am \
diff --git a/libgloss/riscv/Makefile.inc b/libgloss/riscv/Makefile.inc
index 5341038b3..fafc74374 100644
--- a/libgloss/riscv/Makefile.inc
+++ b/libgloss/riscv/Makefile.inc
@@ -69,3 +69,6 @@ multilibtool_LIBRARIES += %D%/libsemihost.a
  %D%/semihost-sys_stat_common.c \
  %D%/semihost-sys_unlink.c \
  %D%/semihost-sys_write.c
+
+includemachinetooldir = $(tooldir)/include/machine
+includemachinetool_DATA = %D%/machine/syscall.h
-- 
2.37.0 (Apple Git-136)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] libgloss: riscv: Install machine/syscall.h
  2022-10-18 19:53 [PATCH] libgloss: riscv: Install machine/syscall.h Simon Cook
@ 2022-10-19 10:47 ` Corinna Vinschen
  2022-10-19 11:04   ` Simon Cook
  0 siblings, 1 reply; 6+ messages in thread
From: Corinna Vinschen @ 2022-10-19 10:47 UTC (permalink / raw)
  To: Simon Cook; +Cc: newlib

Hi Simon,

On Oct 18 21:53, Simon Cook wrote:
> A recent change to the Makefile.in for riscv resulted in the
> machine/syscall.h header not being installed. This updates the file
> to install this file again.
> 
> Signed-off-by: Simon Cook <simon.cook@embecosm.com>
> ---
>  libgloss/Makefile.in        | 68 +++++++++++++++++++++++++------------

This change needs to go into Makefile.am, Makefile.in gets just
recreated from there...


Thanks,
Corinna


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] libgloss: riscv: Install machine/syscall.h
  2022-10-19 10:47 ` Corinna Vinschen
@ 2022-10-19 11:04   ` Simon Cook
  2022-10-19 16:14     ` Corinna Vinschen
  0 siblings, 1 reply; 6+ messages in thread
From: Simon Cook @ 2022-10-19 11:04 UTC (permalink / raw)
  To: newlib

Hi Corinna,

My understanding is Makefile.am includes riscv/Makefile.inc which is the 
only file I manually edited, the Makefile.in changes were just those 
made when I ran automake in the libgloss folder after the Makefile.inc 
change. I'm not quite sure what bit I'm missing adding to Makefile.am 
directly?

Thanks,
Simon

On 19/10/2022 12:47, Corinna Vinschen wrote:
> Hi Simon,
> 
> On Oct 18 21:53, Simon Cook wrote:
>> A recent change to the Makefile.in for riscv resulted in the
>> machine/syscall.h header not being installed. This updates the file
>> to install this file again.
>>
>> Signed-off-by: Simon Cook <simon.cook@embecosm.com>
>> ---
>>   libgloss/Makefile.in        | 68 +++++++++++++++++++++++++------------
> 
> This change needs to go into Makefile.am, Makefile.in gets just
> recreated from there...
> 
> 
> Thanks,
> Corinna
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] libgloss: riscv: Install machine/syscall.h
  2022-10-19 11:04   ` Simon Cook
@ 2022-10-19 16:14     ` Corinna Vinschen
  2022-10-19 19:03       ` Simon Cook
  0 siblings, 1 reply; 6+ messages in thread
From: Corinna Vinschen @ 2022-10-19 16:14 UTC (permalink / raw)
  To: Simon Cook; +Cc: newlib

Hi Simon,

On Oct 19 13:04, Simon Cook wrote:
> Hi Corinna,
> 
> My understanding is Makefile.am includes riscv/Makefile.inc which is the
> only file I manually edited, the Makefile.in changes were just those made
> when I ran automake in the libgloss folder after the Makefile.inc change.

Sorry, my bad, I didn't realise that when glancing at the patch.

Still, unfortunately your patch doesn't apply.  It seems whitespaces,
especially leading TABs, are broken in your patch.  I suspect your
MUA is breaking things.  Maybe you can check this and, if all else fails,
just attach the patch to your mail?


Thanks,
Corinna


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] libgloss: riscv: Install machine/syscall.h
  2022-10-19 16:14     ` Corinna Vinschen
@ 2022-10-19 19:03       ` Simon Cook
  2022-10-20 14:54         ` Corinna Vinschen
  0 siblings, 1 reply; 6+ messages in thread
From: Simon Cook @ 2022-10-19 19:03 UTC (permalink / raw)
  To: newlib, vinschen

[-- Attachment #1: Type: text/plain, Size: 417 bytes --]

Hi Corinna,

On 19/10/2022 18:14, Corinna Vinschen wrote:
> Still, unfortunately your patch doesn't apply.  It seems whitespaces,
> especially leading TABs, are broken in your patch.  I suspect your
> MUA is breaking things.  Maybe you can check this and, if all else fails,
> just attach the patch to your mail?

Sorry about that, I've attached the patch which should apply without 
whitespace errors.

Thanks,
Simon

[-- Attachment #2: 0001-libgloss-riscv-Install-machine-syscall.h.patch --]
[-- Type: text/plain, Size: 7580 bytes --]

From 54f2ef3d64aef67f6fa355833c774df4dcf7f9f8 Mon Sep 17 00:00:00 2001
From: Simon Cook <simon.cook@embecosm.com>
Date: Tue, 18 Oct 2022 21:48:56 +0200
Subject: [PATCH] libgloss: riscv: Install machine/syscall.h

A recent change to the Makefile.in for riscv resulted in the
machine/syscall.h header not being installed. This updates the file
to install this file again.

Signed-off-by: Simon Cook <simon.cook@embecosm.com>
---
 libgloss/Makefile.in        | 68 +++++++++++++++++++++++++------------
 libgloss/riscv/Makefile.inc |  3 ++
 2 files changed, 50 insertions(+), 21 deletions(-)

diff --git a/libgloss/Makefile.in b/libgloss/Makefile.in
index 16723dca1..36ab3fbd3 100644
--- a/libgloss/Makefile.in
+++ b/libgloss/Makefile.in
@@ -212,8 +212,9 @@ am__uninstall_files_from_dir = { \
          $(am__cd) "$$dir" && rm -f $$files; }; \
   }
 am__installdirs = "$(DESTDIR)$(multilibtooldir)" "$(DESTDIR)$(bindir)" \
-	"$(DESTDIR)$(infodir)" "$(DESTDIR)$(includesystooldir)" \
-	"$(DESTDIR)$(includetooldir)" "$(DESTDIR)$(multilibtooldir)"
+	"$(DESTDIR)$(infodir)" "$(DESTDIR)$(includemachinetooldir)" \
+	"$(DESTDIR)$(includesystooldir)" "$(DESTDIR)$(includetooldir)" \
+	"$(DESTDIR)$(multilibtooldir)"
 LIBRARIES = $(multilibtool_LIBRARIES) $(noinst_LIBRARIES)
 ARFLAGS = cru
 AM_V_AR = $(am__v_AR_@AM_V@)
@@ -471,7 +472,8 @@ am__can_run_installinfo = \
     n|no|NO) false;; \
     *) (install-info --version) >/dev/null 2>&1;; \
   esac
-DATA = $(includesystool_DATA) $(includetool_DATA) $(multilibtool_DATA)
+DATA = $(includemachinetool_DATA) $(includesystool_DATA) \
+	$(includetool_DATA) $(multilibtool_DATA)
 RECURSIVE_CLEAN_TARGETS = mostlyclean-recursive clean-recursive	\
   distclean-recursive maintainer-clean-recursive
 am__recursive_targets = \
@@ -610,7 +612,6 @@ pdfdir = @pdfdir@
 prefix = @prefix@
 program_transform_name = @program_transform_name@
 psdir = @psdir@
-runstatedir = @runstatedir@
 sbindir = @sbindir@
 sharedstatedir = @sharedstatedir@
 srcdir = @srcdir@
@@ -791,6 +792,8 @@ TEXINFO_TEX = ../texinfo/texinfo.tex
 @CONFIG_RISCV_TRUE@	riscv/semihost-sys_unlink.c \
 @CONFIG_RISCV_TRUE@	riscv/semihost-sys_write.c
 
+@CONFIG_RISCV_TRUE@includemachinetooldir = $(tooldir)/include/machine
+@CONFIG_RISCV_TRUE@includemachinetool_DATA = riscv/machine/syscall.h
 @CONFIG_WINCE_TRUE@gdbdir = ${dir ${patsubst %/,%,${dir @srcdir@}}}gdb
 @CONFIG_WINCE_TRUE@wince_stub_exe_SOURCES = wince-stub.c
 @CONFIG_WINCE_TRUE@wince_stub_exe_CPPFLAGS = $(AM_CPPFLAGS) -I$(gdbdir)
@@ -2504,6 +2507,27 @@ maintainer-clean-aminfo:
 	  echo " rm -f $$i $$i-[0-9] $$i-[0-9][0-9] $$i_i[0-9] $$i_i[0-9][0-9]"; \
 	  rm -f $$i $$i-[0-9] $$i-[0-9][0-9] $$i_i[0-9] $$i_i[0-9][0-9]; \
 	done
+install-includemachinetoolDATA: $(includemachinetool_DATA)
+	@$(NORMAL_INSTALL)
+	@list='$(includemachinetool_DATA)'; test -n "$(includemachinetooldir)" || list=; \
+	if test -n "$$list"; then \
+	  echo " $(MKDIR_P) '$(DESTDIR)$(includemachinetooldir)'"; \
+	  $(MKDIR_P) "$(DESTDIR)$(includemachinetooldir)" || exit 1; \
+	fi; \
+	for p in $$list; do \
+	  if test -f "$$p"; then d=; else d="$(srcdir)/"; fi; \
+	  echo "$$d$$p"; \
+	done | $(am__base_list) | \
+	while read files; do \
+	  echo " $(INSTALL_DATA) $$files '$(DESTDIR)$(includemachinetooldir)'"; \
+	  $(INSTALL_DATA) $$files "$(DESTDIR)$(includemachinetooldir)" || exit $$?; \
+	done
+
+uninstall-includemachinetoolDATA:
+	@$(NORMAL_UNINSTALL)
+	@list='$(includemachinetool_DATA)'; test -n "$(includemachinetooldir)" || list=; \
+	files=`for p in $$list; do echo $$p; done | sed -e 's|^.*/||'`; \
+	dir='$(DESTDIR)$(includemachinetooldir)'; $(am__uninstall_files_from_dir)
 install-includesystoolDATA: $(includesystool_DATA)
 	@$(NORMAL_INSTALL)
 	@list='$(includesystool_DATA)'; test -n "$(includesystooldir)" || list=; \
@@ -2680,7 +2704,7 @@ all-am: Makefile $(INFO_DEPS) $(LIBRARIES) $(PROGRAMS) $(DATA) \
 		config.h all-local
 installdirs: installdirs-recursive
 installdirs-am:
-	for dir in "$(DESTDIR)$(multilibtooldir)" "$(DESTDIR)$(bindir)" "$(DESTDIR)$(infodir)" "$(DESTDIR)$(includesystooldir)" "$(DESTDIR)$(includetooldir)" "$(DESTDIR)$(multilibtooldir)"; do \
+	for dir in "$(DESTDIR)$(multilibtooldir)" "$(DESTDIR)$(bindir)" "$(DESTDIR)$(infodir)" "$(DESTDIR)$(includemachinetooldir)" "$(DESTDIR)$(includesystooldir)" "$(DESTDIR)$(includetooldir)" "$(DESTDIR)$(multilibtooldir)"; do \
 	  test -z "$$dir" || $(MKDIR_P) "$$dir"; \
 	done
 install: install-recursive
@@ -2750,7 +2774,8 @@ info: info-recursive
 
 info-am: $(INFO_DEPS)
 
-install-data-am: install-includesystoolDATA install-includetoolDATA \
+install-data-am: install-includemachinetoolDATA \
+	install-includesystoolDATA install-includetoolDATA \
 	install-info-am install-multilibtoolDATA \
 	install-multilibtoolLIBRARIES
 
@@ -2893,10 +2918,10 @@ ps: ps-recursive
 ps-am: $(PSS)
 
 uninstall-am: uninstall-binPROGRAMS uninstall-dvi-am uninstall-html-am \
-	uninstall-includesystoolDATA uninstall-includetoolDATA \
-	uninstall-info-am uninstall-multilibtoolDATA \
-	uninstall-multilibtoolLIBRARIES uninstall-pdf-am \
-	uninstall-ps-am
+	uninstall-includemachinetoolDATA uninstall-includesystoolDATA \
+	uninstall-includetoolDATA uninstall-info-am \
+	uninstall-multilibtoolDATA uninstall-multilibtoolLIBRARIES \
+	uninstall-pdf-am uninstall-ps-am
 
 .MAKE: $(am__recursive_targets) all check-am install-am install-strip
 
@@ -2910,17 +2935,18 @@ uninstall-am: uninstall-binPROGRAMS uninstall-dvi-am uninstall-html-am \
 	html-am info info-am install install-am install-binPROGRAMS \
 	install-data install-data-am install-dvi install-dvi-am \
 	install-exec install-exec-am install-exec-local install-html \
-	install-html-am install-includesystoolDATA \
-	install-includetoolDATA install-info install-info-am \
-	install-man install-multilibtoolDATA \
-	install-multilibtoolLIBRARIES install-pdf install-pdf-am \
-	install-ps install-ps-am install-strip installcheck \
-	installcheck-am installdirs installdirs-am maintainer-clean \
-	maintainer-clean-aminfo maintainer-clean-generic \
-	maintainer-clean-local mostlyclean mostlyclean-aminfo \
-	mostlyclean-compile mostlyclean-generic mostlyclean-local pdf \
-	pdf-am ps ps-am tags tags-am uninstall uninstall-am \
-	uninstall-binPROGRAMS uninstall-dvi-am uninstall-html-am \
+	install-html-am install-includemachinetoolDATA \
+	install-includesystoolDATA install-includetoolDATA \
+	install-info install-info-am install-man \
+	install-multilibtoolDATA install-multilibtoolLIBRARIES \
+	install-pdf install-pdf-am install-ps install-ps-am \
+	install-strip installcheck installcheck-am installdirs \
+	installdirs-am maintainer-clean maintainer-clean-aminfo \
+	maintainer-clean-generic maintainer-clean-local mostlyclean \
+	mostlyclean-aminfo mostlyclean-compile mostlyclean-generic \
+	mostlyclean-local pdf pdf-am ps ps-am tags tags-am uninstall \
+	uninstall-am uninstall-binPROGRAMS uninstall-dvi-am \
+	uninstall-html-am uninstall-includemachinetoolDATA \
 	uninstall-includesystoolDATA uninstall-includetoolDATA \
 	uninstall-info-am uninstall-multilibtoolDATA \
 	uninstall-multilibtoolLIBRARIES uninstall-pdf-am \
diff --git a/libgloss/riscv/Makefile.inc b/libgloss/riscv/Makefile.inc
index 5341038b3..fafc74374 100644
--- a/libgloss/riscv/Makefile.inc
+++ b/libgloss/riscv/Makefile.inc
@@ -69,3 +69,6 @@ multilibtool_LIBRARIES += %D%/libsemihost.a
 	%D%/semihost-sys_stat_common.c \
 	%D%/semihost-sys_unlink.c \
 	%D%/semihost-sys_write.c
+
+includemachinetooldir = $(tooldir)/include/machine
+includemachinetool_DATA = %D%/machine/syscall.h
-- 
2.37.0 (Apple Git-136)


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] libgloss: riscv: Install machine/syscall.h
  2022-10-19 19:03       ` Simon Cook
@ 2022-10-20 14:54         ` Corinna Vinschen
  0 siblings, 0 replies; 6+ messages in thread
From: Corinna Vinschen @ 2022-10-20 14:54 UTC (permalink / raw)
  To: Simon Cook; +Cc: newlib

On Oct 19 21:03, Simon Cook wrote:
> Hi Corinna,
> 
> On 19/10/2022 18:14, Corinna Vinschen wrote:
> > Still, unfortunately your patch doesn't apply.  It seems whitespaces,
> > especially leading TABs, are broken in your patch.  I suspect your
> > MUA is breaking things.  Maybe you can check this and, if all else fails,
> > just attach the patch to your mail?
> 
> Sorry about that, I've attached the patch which should apply without
> whitespace errors.

That worked, pushed.


Thanks,
Corinna


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-10-20 14:54 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-18 19:53 [PATCH] libgloss: riscv: Install machine/syscall.h Simon Cook
2022-10-19 10:47 ` Corinna Vinschen
2022-10-19 11:04   ` Simon Cook
2022-10-19 16:14     ` Corinna Vinschen
2022-10-19 19:03       ` Simon Cook
2022-10-20 14:54         ` Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).