aboutsummaryrefslogtreecommitdiffstats
path: root/main/apk-tools/0002-print-use-stdout-instead-of-stderr-for-logging-and-p.patch
blob: 340ad079e5126fe2280ad25d1cecda6c719b8c0e (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
From 517378721855280d2e23d25d7529e6b9cbae9136 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timo=20Ter=C3=A4s?= <timo.teras@iki.fi>
Date: Fri, 11 Oct 2013 14:52:34 +0300
Subject: [PATCH 2/3] print: use stdout instead of stderr for logging and
 progress

stdout is the proper place for it. this also fixes the progress
bar in musl, which seems to not support using line buffering for
stderr.
---
 src/apk.c   |  1 -
 src/print.c | 20 ++++++++++----------
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/src/apk.c b/src/apk.c
index 8744de6..8a3c7ef 100644
--- a/src/apk.c
+++ b/src/apk.c
@@ -266,7 +266,6 @@ static void on_sigwinch(int s)
 
 static void setup_terminal(void)
 {
-	setvbuf(stderr, NULL, _IOLBF, BUFSIZ);
 	signal(SIGWINCH, on_sigwinch);
 	signal(SIGPIPE, SIG_IGN);
 }
diff --git a/src/print.c b/src/print.c
index 6d00064..aafe2ef 100644
--- a/src/print.c
+++ b/src/print.c
@@ -35,7 +35,7 @@ int apk_get_screen_width(void)
 
 	if (apk_screen_width == 0) {
 		apk_screen_width = 50;
-		if (ioctl(STDERR_FILENO, TIOCGWINSZ, &w) == 0 &&
+		if (ioctl(STDOUT_FILENO, TIOCGWINSZ, &w) == 0 &&
 		    w.ws_col > 50)
 			apk_screen_width = w.ws_col;
 	}
@@ -77,14 +77,14 @@ void apk_print_progress(size_t done, size_t total)
 	last_percent = percent;
 	apk_progress_force = 0;
 
-	fprintf(stderr, "\e7%3i%% [", percent);
+	fprintf(stdout, "\e7%3i%% [", percent);
 	for (i = 0; i < bar; i++)
-		fputc('#', stderr);
+		fputc('#', stdout);
 	for (; i < bar_width; i++)
-		fputc(' ', stderr);
-	fputc(']', stderr);
-	fflush(stderr);
-	fputs("\e8\e[0K", stderr);
+		fputc(' ', stdout);
+	fputc(']', stdout);
+	fflush(stdout);
+	fputs("\e8\e[0K", stdout);
 }
 
 int apk_print_indented(struct apk_indent *i, apk_blob_t blob)
@@ -144,11 +144,11 @@ void apk_log(const char *prefix, const char *format, ...)
 	va_list va;
 
 	if (prefix != NULL)
-		fprintf(stderr, "%s", prefix);
+		fprintf(stdout, "%s", prefix);
 	va_start(va, format);
-	vfprintf(stderr, format, va);
+	vfprintf(stdout, format, va);
 	va_end(va);
-	fprintf(stderr, "\n");
+	fprintf(stdout, "\n");
 	apk_progress_force = 1;
 }
 
-- 
1.8.5.1