public inbox for cygwin-patches@cygwin.com
 help / color / mirror / Atom feed
* [PATCH] Compile sigfe.s with CFLAGS
@ 2015-02-19 13:21 Jon TURNEY
  2015-02-19 13:29 ` Corinna Vinschen
  0 siblings, 1 reply; 2+ messages in thread
From: Jon TURNEY @ 2015-02-19 13:21 UTC (permalink / raw)
  To: Cygwin Patches

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


CFLAGS isn't used when compiling the generated file sigfe.s, so if that 
contains -g, it lacks source debugging information.


[-- Attachment #2: sigfe_cflags.patch --]
[-- Type: text/plain, Size: 570 bytes --]

2015-02-19  Jon TURNEY  <jon.turney@dronecode.org.uk>

	* Makefile.in (sigfe.o): Use CFLAGS.

Index: cygwin/Makefile.in
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/Makefile.in,v
retrieving revision 1.278
diff -u -u -p -r1.278 Makefile.in
--- cygwin/Makefile.in	28 Jan 2015 11:43:06 -0000	1.278
+++ cygwin/Makefile.in	19 Feb 2015 13:12:11 -0000
@@ -710,7 +710,7 @@ sigfe.s: $(DEF_FILE)
 	[ -s $@ ] && touch $@
 
 sigfe.o: sigfe.s
-	$(CC) -c -o $@ $<
+	$(CC) ${CFLAGS} -c -o $@ $<
 
 ctags: CTAGS
 tags:  CTAGS

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

* Re: [PATCH] Compile sigfe.s with CFLAGS
  2015-02-19 13:21 [PATCH] Compile sigfe.s with CFLAGS Jon TURNEY
@ 2015-02-19 13:29 ` Corinna Vinschen
  0 siblings, 0 replies; 2+ messages in thread
From: Corinna Vinschen @ 2015-02-19 13:29 UTC (permalink / raw)
  To: cygwin-patches

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

On Feb 19 13:21, Jon TURNEY wrote:
> 	* Makefile.in (sigfe.o): Use CFLAGS.

Please apply.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

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

end of thread, other threads:[~2015-02-19 13:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-19 13:21 [PATCH] Compile sigfe.s with CFLAGS Jon TURNEY
2015-02-19 13:29 ` 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).