all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#70716: flymake-cc creates `-.o` files
@ 2024-05-02 13:10 Mattias Engdegård
  2024-05-02 13:13 ` João Távora
  0 siblings, 1 reply; 13+ messages in thread
From: Mattias Engdegård @ 2024-05-02 13:10 UTC (permalink / raw)
  To: 70716; +Cc: João Távora

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

`flymake-cc-use-special-make-target`, the standard value for `flymake-cc-command`, generates a command like

  make check-syntax CHK_SOURCES="-x c -c -"

which causes the compiler (at least Clang) to create the file `-.o` that is never cleaned up and whose name is mysterious and awkward to remove.

What about the patch below?


[-- Attachment #2: flymake-cc-null-output.diff --]
[-- Type: application/octet-stream, Size: 637 bytes --]

diff --git a/lisp/progmodes/flymake-cc.el b/lisp/progmodes/flymake-cc.el
index 60e7da5d617..01f463454e9 100644
--- a/lisp/progmodes/flymake-cc.el
+++ b/lisp/progmodes/flymake-cc.el
@@ -95,9 +95,10 @@ flymake-cc-use-special-make-target
   (unless (executable-find "make") (error "Make not found"))
   `("make"
     "check-syntax"
-    ,(format "CHK_SOURCES=-x %s -c -"
+    ,(format "CHK_SOURCES=-x %s -c - -o %s"
              (cond ((derived-mode-p 'c++-mode) "c++")
-                   (t "c")))))
+                   (t "c"))
+             null-device)))
 
 (defvar-local flymake-cc--proc nil "Internal variable for `flymake-cc'.")
 

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

* bug#70716: flymake-cc creates `-.o` files
  2024-05-02 13:10 bug#70716: flymake-cc creates `-.o` files Mattias Engdegård
@ 2024-05-02 13:13 ` João Távora
  2024-05-02 14:00   ` Mattias Engdegård
  2024-05-18  8:28   ` Eli Zaretskii
  0 siblings, 2 replies; 13+ messages in thread
From: João Távora @ 2024-05-02 13:13 UTC (permalink / raw)
  To: Mattias Engdegård, Spencer Baugh; +Cc: 70716

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

On Thu, May 2, 2024 at 2:10 PM Mattias Engdegård <
mattias.engdegard@gmail.com> wrote:

> `flymake-cc-use-special-make-target`, the standard value for
> `flymake-cc-command`, generates a command like
>
>   make check-syntax CHK_SOURCES="-x c -c -"
>
> which causes the compiler (at least Clang) to create the file `-.o` that
> is never cleaned up and whose name is mysterious and awkward to remove.
>
> What about the patch below?
>

Copying in Spencer Baugh, the new Flymake maintainer.

João

[-- Attachment #2: Type: text/html, Size: 898 bytes --]

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

* bug#70716: flymake-cc creates `-.o` files
  2024-05-02 13:13 ` João Távora
@ 2024-05-02 14:00   ` Mattias Engdegård
  2024-05-04 11:38     ` Eli Zaretskii
  2024-05-18  8:28   ` Eli Zaretskii
  1 sibling, 1 reply; 13+ messages in thread
From: Mattias Engdegård @ 2024-05-02 14:00 UTC (permalink / raw)
  To: João Távora; +Cc: sbaugh, 70716

2 maj 2024 kl. 15.13 skrev João Távora <joaotavora@gmail.com>:

> Copying in Spencer Baugh, the new Flymake maintainer. 

Would either of you please update the `Maintainer:` lines in lisp/progmodes/flymake*.el then?






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

* bug#70716: flymake-cc creates `-.o` files
  2024-05-02 14:00   ` Mattias Engdegård
@ 2024-05-04 11:38     ` Eli Zaretskii
  2024-05-18  8:33       ` Eli Zaretskii
  0 siblings, 1 reply; 13+ messages in thread
From: Eli Zaretskii @ 2024-05-04 11:38 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: sbaugh, 70716, joaotavora

> Cc: sbaugh@catern.com, 70716@debbugs.gnu.org
> From: Mattias Engdegård <mattias.engdegard@gmail.com>
> Date: Thu, 2 May 2024 16:00:08 +0200
> 
> 2 maj 2024 kl. 15.13 skrev João Távora <joaotavora@gmail.com>:
> 
> > Copying in Spencer Baugh, the new Flymake maintainer. 
> 
> Would either of you please update the `Maintainer:` lines in lisp/progmodes/flymake*.el then?

Spencer, is it okay to mention you there as the maintainer of Flymake?





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

* bug#70716: flymake-cc creates `-.o` files
  2024-05-02 13:13 ` João Távora
  2024-05-02 14:00   ` Mattias Engdegård
@ 2024-05-18  8:28   ` Eli Zaretskii
  2024-06-30  5:45     ` Stefan Kangas
  2024-08-28 17:01     ` sbaugh
  1 sibling, 2 replies; 13+ messages in thread
From: Eli Zaretskii @ 2024-05-18  8:28 UTC (permalink / raw)
  To: João Távora, sbaugh; +Cc: mattias.engdegard, 70716

> Cc: 70716@debbugs.gnu.org
> From: João Távora <joaotavora@gmail.com>
> Date: Thu, 2 May 2024 14:13:49 +0100
> 
> On Thu, May 2, 2024 at 2:10 PM Mattias Engdegård <mattias.engdegard@gmail.com> wrote:
> 
>  `flymake-cc-use-special-make-target`, the standard value for `flymake-cc-command`, generates a
>  command like
> 
>    make check-syntax CHK_SOURCES="-x c -c -"
> 
>  which causes the compiler (at least Clang) to create the file `-.o` that is never cleaned up and whose
>  name is mysterious and awkward to remove.
> 
>  What about the patch below?
> 
>  
> Copying in Spencer Baugh, the new Flymake maintainer. 

Spencer, any comments or suggestions or patches?

Thanks.





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

* bug#70716: flymake-cc creates `-.o` files
  2024-05-04 11:38     ` Eli Zaretskii
@ 2024-05-18  8:33       ` Eli Zaretskii
  0 siblings, 0 replies; 13+ messages in thread
From: Eli Zaretskii @ 2024-05-18  8:33 UTC (permalink / raw)
  To: sbaugh; +Cc: mattias.engdegard, 70716, joaotavora

Ping!

> Cc: sbaugh@catern.com, 70716@debbugs.gnu.org, joaotavora@gmail.com
> Date: Sat, 04 May 2024 14:38:39 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> 
> > Cc: sbaugh@catern.com, 70716@debbugs.gnu.org
> > From: Mattias Engdegård <mattias.engdegard@gmail.com>
> > Date: Thu, 2 May 2024 16:00:08 +0200
> > 
> > 2 maj 2024 kl. 15.13 skrev João Távora <joaotavora@gmail.com>:
> > 
> > > Copying in Spencer Baugh, the new Flymake maintainer. 
> > 
> > Would either of you please update the `Maintainer:` lines in lisp/progmodes/flymake*.el then?
> 
> Spencer, is it okay to mention you there as the maintainer of Flymake?
> 
> 
> 
> 





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

* bug#70716: flymake-cc creates `-.o` files
  2024-05-18  8:28   ` Eli Zaretskii
@ 2024-06-30  5:45     ` Stefan Kangas
  2024-07-20 10:10       ` Eli Zaretskii
  2024-08-28 17:01     ` sbaugh
  1 sibling, 1 reply; 13+ messages in thread
From: Stefan Kangas @ 2024-06-30  5:45 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: sbaugh, mattias.engdegard, João Távora, 70716

Eli Zaretskii <eliz@gnu.org> writes:

>> Cc: 70716@debbugs.gnu.org
>> From: João Távora <joaotavora@gmail.com>
>> Date: Thu, 2 May 2024 14:13:49 +0100
>>
>> On Thu, May 2, 2024 at 2:10 PM Mattias Engdegård <mattias.engdegard@gmail.com> wrote:
>>
>>  `flymake-cc-use-special-make-target`, the standard value for `flymake-cc-command`, generates a
>>  command like
>>
>>    make check-syntax CHK_SOURCES="-x c -c -"
>>
>>  which causes the compiler (at least Clang) to create the file `-.o` that is never cleaned up and whose
>>  name is mysterious and awkward to remove.
>>
>>  What about the patch below?
>>
>>
>> Copying in Spencer Baugh, the new Flymake maintainer.
>
> Spencer, any comments or suggestions or patches?
>
> Thanks.

Ping.





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

* bug#70716: flymake-cc creates `-.o` files
  2024-06-30  5:45     ` Stefan Kangas
@ 2024-07-20 10:10       ` Eli Zaretskii
  2024-08-04  7:55         ` Eli Zaretskii
  0 siblings, 1 reply; 13+ messages in thread
From: Eli Zaretskii @ 2024-07-20 10:10 UTC (permalink / raw)
  To: Stefan Kangas, sbaugh; +Cc: mattias.engdegard, joaotavora, 70716

Ping! Spencer, would you please chime in?

> From: Stefan Kangas <stefankangas@gmail.com>
> Date: Sat, 29 Jun 2024 22:45:50 -0700
> Cc: João Távora <joaotavora@gmail.com>, sbaugh@catern.com, 
> 	mattias.engdegard@gmail.com, 70716@debbugs.gnu.org
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> Cc: 70716@debbugs.gnu.org
> >> From: João Távora <joaotavora@gmail.com>
> >> Date: Thu, 2 May 2024 14:13:49 +0100
> >>
> >> On Thu, May 2, 2024 at 2:10 PM Mattias Engdegård <mattias.engdegard@gmail.com> wrote:
> >>
> >>  `flymake-cc-use-special-make-target`, the standard value for `flymake-cc-command`, generates a
> >>  command like
> >>
> >>    make check-syntax CHK_SOURCES="-x c -c -"
> >>
> >>  which causes the compiler (at least Clang) to create the file `-.o` that is never cleaned up and whose
> >>  name is mysterious and awkward to remove.
> >>
> >>  What about the patch below?
> >>
> >>
> >> Copying in Spencer Baugh, the new Flymake maintainer.
> >
> > Spencer, any comments or suggestions or patches?
> >
> > Thanks.
> 
> Ping.
> 





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

* bug#70716: flymake-cc creates `-.o` files
  2024-07-20 10:10       ` Eli Zaretskii
@ 2024-08-04  7:55         ` Eli Zaretskii
  2024-08-17  8:22           ` Eli Zaretskii
  0 siblings, 1 reply; 13+ messages in thread
From: Eli Zaretskii @ 2024-08-04  7:55 UTC (permalink / raw)
  To: sbaugh; +Cc: mattias.engdegard, stefankangas, 70716, joaotavora

Ping! Ping!  Spencer, please respond.

> Cc: mattias.engdegard@gmail.com, joaotavora@gmail.com, 70716@debbugs.gnu.org
> Date: Sat, 20 Jul 2024 13:10:52 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> 
> Ping! Spencer, would you please chime in?
> 
> > From: Stefan Kangas <stefankangas@gmail.com>
> > Date: Sat, 29 Jun 2024 22:45:50 -0700
> > Cc: João Távora <joaotavora@gmail.com>, sbaugh@catern.com, 
> > 	mattias.engdegard@gmail.com, 70716@debbugs.gnu.org
> > 
> > Eli Zaretskii <eliz@gnu.org> writes:
> > 
> > >> Cc: 70716@debbugs.gnu.org
> > >> From: João Távora <joaotavora@gmail.com>
> > >> Date: Thu, 2 May 2024 14:13:49 +0100
> > >>
> > >> On Thu, May 2, 2024 at 2:10 PM Mattias Engdegård <mattias.engdegard@gmail.com> wrote:
> > >>
> > >>  `flymake-cc-use-special-make-target`, the standard value for `flymake-cc-command`, generates a
> > >>  command like
> > >>
> > >>    make check-syntax CHK_SOURCES="-x c -c -"
> > >>
> > >>  which causes the compiler (at least Clang) to create the file `-.o` that is never cleaned up and whose
> > >>  name is mysterious and awkward to remove.
> > >>
> > >>  What about the patch below?
> > >>
> > >>
> > >> Copying in Spencer Baugh, the new Flymake maintainer.
> > >
> > > Spencer, any comments or suggestions or patches?
> > >
> > > Thanks.
> > 
> > Ping.
> > 
> 
> 
> 
> 





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

* bug#70716: flymake-cc creates `-.o` files
  2024-08-04  7:55         ` Eli Zaretskii
@ 2024-08-17  8:22           ` Eli Zaretskii
  2024-08-28 17:01             ` Spencer Baugh
  0 siblings, 1 reply; 13+ messages in thread
From: Eli Zaretskii @ 2024-08-17  8:22 UTC (permalink / raw)
  To: sbaugh; +Cc: joaotavora, mattias.engdegard, stefankangas, 70716

Ping! Ping! Ping!  Spencer, are you there?

> Cc: mattias.engdegard@gmail.com, stefankangas@gmail.com, 70716@debbugs.gnu.org,
>  joaotavora@gmail.com
> Date: Sun, 04 Aug 2024 10:55:48 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> 
> Ping! Ping!  Spencer, please respond.
> 
> > Cc: mattias.engdegard@gmail.com, joaotavora@gmail.com, 70716@debbugs.gnu.org
> > Date: Sat, 20 Jul 2024 13:10:52 +0300
> > From: Eli Zaretskii <eliz@gnu.org>
> > 
> > Ping! Spencer, would you please chime in?
> > 
> > > From: Stefan Kangas <stefankangas@gmail.com>
> > > Date: Sat, 29 Jun 2024 22:45:50 -0700
> > > Cc: João Távora <joaotavora@gmail.com>, sbaugh@catern.com, 
> > > 	mattias.engdegard@gmail.com, 70716@debbugs.gnu.org
> > > 
> > > Eli Zaretskii <eliz@gnu.org> writes:
> > > 
> > > >> Cc: 70716@debbugs.gnu.org
> > > >> From: João Távora <joaotavora@gmail.com>
> > > >> Date: Thu, 2 May 2024 14:13:49 +0100
> > > >>
> > > >> On Thu, May 2, 2024 at 2:10 PM Mattias Engdegård <mattias.engdegard@gmail.com> wrote:
> > > >>
> > > >>  `flymake-cc-use-special-make-target`, the standard value for `flymake-cc-command`, generates a
> > > >>  command like
> > > >>
> > > >>    make check-syntax CHK_SOURCES="-x c -c -"
> > > >>
> > > >>  which causes the compiler (at least Clang) to create the file `-.o` that is never cleaned up and whose
> > > >>  name is mysterious and awkward to remove.
> > > >>
> > > >>  What about the patch below?
> > > >>
> > > >>
> > > >> Copying in Spencer Baugh, the new Flymake maintainer.
> > > >
> > > > Spencer, any comments or suggestions or patches?
> > > >
> > > > Thanks.
> > > 
> > > Ping.
> > > 
> > 
> > 
> > 
> > 
> 
> 
> 
> 





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

* bug#70716: flymake-cc creates `-.o` files
  2024-05-18  8:28   ` Eli Zaretskii
  2024-06-30  5:45     ` Stefan Kangas
@ 2024-08-28 17:01     ` sbaugh
  2024-08-28 17:50       ` Mattias Engdegård
  1 sibling, 1 reply; 13+ messages in thread
From: sbaugh @ 2024-08-28 17:01 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: mattias.engdegard, João Távora, 70716

Mattias Engdegård <mattias.engdegard@gmail.com> writes:
> `flymake-cc-use-special-make-target`, the standard value for `flymake-cc-command`, generates a command like
>
>   make check-syntax CHK_SOURCES="-x c -c -"
>
> which causes the compiler (at least Clang) to create the file `-.o` that is never cleaned up and whose name is mysterious and awkward to remove.
>
> What about the patch below?

(info "(flymake) Example---Configuring a tool called via make")

mentions a check-syntax target looking like:

check-syntax:
	gcc -o /dev/null -S ${CHK_SOURCES} || true

So it seems like it would be better to add this -o /dev/null in your
Makefile.

That also allows you to run, e.g.,

  make check-syntax CHK_SOURCES="file1.c file2.c"

from a shell or M-x compile.

So I think this is a bug in your Makefile, not flymake-cc.

But, also, probably the docstring of flymake-cc-use-special-make-target
should be updated to link to the manual.  And probably the manual should
mention that CHK_SOURCES can be an explicit list of files too.

BTW, the CHK_SOURCES target mentioned by flymake is, AFAICT from web
searches, completely unique to flymake.  I wonder if there's a more
standard alternative... but we probably can't change the default of
flymake-cc-command anyway.





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

* bug#70716: flymake-cc creates `-.o` files
  2024-08-17  8:22           ` Eli Zaretskii
@ 2024-08-28 17:01             ` Spencer Baugh
  0 siblings, 0 replies; 13+ messages in thread
From: Spencer Baugh @ 2024-08-28 17:01 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: joaotavora, mattias.engdegard, stefankangas, 70716

Eli Zaretskii <eliz@gnu.org> writes:
> Ping! Ping! Ping!  Spencer, are you there?

My apologies, I had an issue causing me to not receive these emails.

I think it would be best to list sbaugh@janestreet.com as the Flymake
maintainer, since I'm more likely to receive such emails.





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

* bug#70716: flymake-cc creates `-.o` files
  2024-08-28 17:01     ` sbaugh
@ 2024-08-28 17:50       ` Mattias Engdegård
  0 siblings, 0 replies; 13+ messages in thread
From: Mattias Engdegård @ 2024-08-28 17:50 UTC (permalink / raw)
  To: Spencer Baugh; +Cc: Eli Zaretskii, 70716-done, João Távora

28 aug. 2024 kl. 19.01 skrev sbaugh@catern.com:

> check-syntax:
> 	gcc -o /dev/null -S ${CHK_SOURCES} || true
> 
> So it seems like it would be better to add this -o /dev/null in your
> Makefile.

Actually not my Makefile but that of flymake-tests.
Anyway, now done on master, thank you!






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

end of thread, other threads:[~2024-08-28 17:50 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-02 13:10 bug#70716: flymake-cc creates `-.o` files Mattias Engdegård
2024-05-02 13:13 ` João Távora
2024-05-02 14:00   ` Mattias Engdegård
2024-05-04 11:38     ` Eli Zaretskii
2024-05-18  8:33       ` Eli Zaretskii
2024-05-18  8:28   ` Eli Zaretskii
2024-06-30  5:45     ` Stefan Kangas
2024-07-20 10:10       ` Eli Zaretskii
2024-08-04  7:55         ` Eli Zaretskii
2024-08-17  8:22           ` Eli Zaretskii
2024-08-28 17:01             ` Spencer Baugh
2024-08-28 17:01     ` sbaugh
2024-08-28 17:50       ` Mattias Engdegård

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.