9front - general discussion about 9front
 help / color / mirror / Atom feed
* [9front] ape/mkstemp: Include <bsd.h>
@ 2022-01-04 19:47 Benjamin Riefenstahl
  2022-01-07 14:46 ` [9front] Re: ape/mkstemp: Set define (was: Include <bsd.h>) Benjamin Riefenstahl
  0 siblings, 1 reply; 2+ messages in thread
From: Benjamin Riefenstahl @ 2022-01-04 19:47 UTC (permalink / raw)
  To: 9front

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

ape/mkstemp.c currently uses mktemp() without a prototype.

[-- Attachment #2.1: Type: text/plain, Size: 308 bytes --]

from postmaster@4ess:
The following attachment had content that we can't
prove to be harmless.  To avoid possible automatic
execution, we changed the content headers.
The original header was:

	Content-Type: text/x-diff
	Content-Disposition: inline;
 filename=0002-ape-mkstemp-include-header-for-mktemp.patch

[-- Attachment #2.2: 0002-ape-mkstemp-include-header-for-mktemp.patch.suspect --]
[-- Type: application/octet-stream, Size: 524 bytes --]

From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
Date: Fri, 10 Dec 2021 20:47:09 +0000
Subject: [PATCH] ape/mkstemp: include header for mktemp

---
diff 9a027a77a0d407d542dbb49be398a474b10deb2b dd7ad4725c5dfb26ad44e8e4206116f96f054ba4
--- a/sys/src/ape/lib/ap/gen/mkstemp.c	Fri Dec 10 21:44:26 2021
+++ b/sys/src/ape/lib/ap/gen/mkstemp.c	Fri Dec 10 21:47:09 2021
@@ -1,3 +1,6 @@
+#define _BSD_EXTENSION 1
+#include <bsd.h> /* mktemp */
+
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>

-- 
1.0


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

* [9front] Re: ape/mkstemp: Set define (was: Include <bsd.h>)
  2022-01-04 19:47 [9front] ape/mkstemp: Include <bsd.h> Benjamin Riefenstahl
@ 2022-01-07 14:46 ` Benjamin Riefenstahl
  0 siblings, 0 replies; 2+ messages in thread
From: Benjamin Riefenstahl @ 2022-01-07 14:46 UTC (permalink / raw)
  To: 9front

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

Hi all,

The recent commit 1a90e04c0 "ape: export mktemp() in <stdlib.h> when
_BSD_SOURCE is set" reduces the need here to setting that define.

Thanks, benny

[-- Attachment #2.1: Type: text/plain, Size: 272 bytes --]

from postmaster@4ess:
The following attachment had content that we can't
prove to be harmless.  To avoid possible automatic
execution, we changed the content headers.
The original header was:

	Content-Type: text/x-diff
	Content-Disposition: inline; filename=mkstemp.patch

[-- Attachment #2.2: mkstemp.patch.suspect --]
[-- Type: application/octet-stream, Size: 511 bytes --]

From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
Date: Fri, 10 Dec 2021 20:47:09 +0000
Subject: [PATCH] ape/mkstemp: set define for mktemp prototype

---
diff 70edb7fbae4f0a38593c0dc1f936838adc6861ac 45a0f394b723a53a1c5d5b0e266342e0c38a8643
--- a/sys/src/ape/lib/ap/gen/mkstemp.c	Fri Jan  7 02:43:52 2022
+++ b/sys/src/ape/lib/ap/gen/mkstemp.c	Fri Dec 10 21:47:09 2021
@@ -1,3 +1,4 @@
+#define _BSD_SOURCE 1 /* make mktemp visible */
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>


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

end of thread, other threads:[~2022-01-07 16:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-04 19:47 [9front] ape/mkstemp: Include <bsd.h> Benjamin Riefenstahl
2022-01-07 14:46 ` [9front] Re: ape/mkstemp: Set define (was: Include <bsd.h>) Benjamin Riefenstahl

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 NNTP newsgroup(s).