Get rid of err.h as it is not portable
authorAaron Marcher <me@drkhsh.at>
Wed, 28 Mar 2018 16:26:56 +0000 (18:26 +0200)
committerAaron Marcher <me@drkhsh.at>
Wed, 28 Mar 2018 16:26:56 +0000 (18:26 +0200)
Replace warn() and warnx() with fprintf() and add <stdio.h> where
necessary.

13 files changed:
components/battery.c
components/disk.c
components/hostname.c
components/ip.c
components/keyboard_indicators.c
components/load_avg.c
components/num_files.c
components/run_command.c
components/swap.c
components/user.c
components/volume.c
components/wifi.c
util.c

index aef5b5f3b2131ba4f5d34f35f1f8098e0a8121bb..cd1169ddb0509a523cc4386d6154d9e9673cb0a5 100644 (file)
@@ -1,5 +1,4 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
 #include <stdio.h>
 #if defined(__linux__)
 #include <limits.h>
@@ -29,12 +28,12 @@ battery_perc(const char *bat)
 
        fd = open("/dev/apm", O_RDONLY);
        if (fd < 0) {
-               warn("Failed to open file /dev/apm");
+               fprintf(stderr, "Failed to open file /dev/apm");
                return NULL;
        }
 
        if (ioctl(fd, APM_IOC_GETPOWER, &apm_info) < 0) {
-               warn("Failed to get battery info");
+               fprintf(stderr, "Failed to get battery info");
                close(fd);
                return NULL;
        }
index 90a8e0bcc850a7b446a5e6d6d780245b512d9811..3d8140e64e9cb1c51169aef0fec82be9b528c6dc 100644 (file)
@@ -1,5 +1,4 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
 #include <stdio.h>
 #include <sys/statvfs.h>
 
@@ -11,7 +10,7 @@ disk_free(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               warn("Failed to get filesystem info");
+               fprintf(stderr, "Failed to get filesystem info");
                return NULL;
        }
 
@@ -25,7 +24,7 @@ disk_perc(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               warn("Failed to get filesystem info");
+               fprintf(stderr, "Failed to get filesystem info");
                return NULL;
        }
 
@@ -40,7 +39,7 @@ disk_total(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               warn("Failed to get filesystem info");
+               fprintf(stderr, "Failed to get filesystem info");
                return NULL;
        }
 
@@ -53,7 +52,7 @@ disk_used(const char *mnt)
        struct statvfs fs;
 
        if (statvfs(mnt, &fs) < 0) {
-               warn("Failed to get filesystem info");
+               fprintf(stderr, "Failed to get filesystem info");
                return NULL;
        }
 
index aed77a61c34a88230efd1810f6b8cf7b3ccb7674..45dbb5bcb0a3bb97e39c6d0bee9fcfe5109aec52 100644 (file)
@@ -1,5 +1,5 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
+#include <stdio.h>
 #include <unistd.h>
 
 #include "../util.h"
@@ -8,7 +8,7 @@ const char *
 hostname(void)
 {
        if (gethostname(buf, sizeof(buf)) == -1) {
-               warn("hostname");
+               fprintf(stderr, "gethostbyname failed");
                return NULL;
        }
 
index 25071e46f2f93c4cdc63dfd2168aa27648e1fc52..c46ec9e685a4c09b78a28fe671dac7befb81fbe1 100644 (file)
@@ -1,6 +1,5 @@
 /* See LICENSE file for copyright and license details. */
 #if defined(__linux__)
-#include <err.h>
 #include <ifaddrs.h>
 #include <netdb.h>
 #include <stdio.h>
@@ -16,7 +15,7 @@ ipv4(const char *iface)
        char host[NI_MAXHOST];
 
        if (getifaddrs(&ifaddr) == -1) {
-               warn("Failed to get IPv4 address for interface %s", iface);
+               fprintf(stderr, "Failed to get IPv4 address for interface %s", iface);
                return NULL;
        }
 
@@ -27,7 +26,7 @@ ipv4(const char *iface)
                s = getnameinfo(ifa->ifa_addr, sizeof(struct sockaddr_in), host, NI_MAXHOST, NULL, 0, NI_NUMERICHOST);
                if ((strcmp(ifa->ifa_name, iface) == 0) && (ifa->ifa_addr->sa_family == AF_INET)) {
                        if (s != 0) {
-                               warnx("Failed to get IPv4 address for interface %s", iface);
+                               fprintf(stderr, "Failed to get IPv4 address for interface %s", iface);
                                return NULL;
                        }
                        return bprintf("%s", host);
@@ -47,7 +46,7 @@ ipv6(const char *iface)
        char host[NI_MAXHOST];
 
        if (getifaddrs(&ifaddr) == -1) {
-               warn("Failed to get IPv6 address for interface %s", iface);
+               fprintf(stderr, "Failed to get IPv6 address for interface %s", iface);
                return NULL;
        }
 
@@ -58,7 +57,7 @@ ipv6(const char *iface)
                s = getnameinfo(ifa->ifa_addr, sizeof(struct sockaddr_in6), host, NI_MAXHOST, NULL, 0, NI_NUMERICHOST);
                if ((strcmp(ifa->ifa_name, iface) == 0) && (ifa->ifa_addr->sa_family == AF_INET6)) {
                        if (s != 0) {
-                               warnx("Failed to get IPv6 address for interface %s", iface);
+                               fprintf(stderr, "Failed to get IPv6 address for interface %s", iface);
                                return NULL;
                        }
                        return bprintf("%s", host);
index b7713b628279ad30bb49afd018c136f9d1c0c6c7..47b52c66a4695f0262b584d31ab9cd63e00f1342 100644 (file)
@@ -1,5 +1,5 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
+#include <stdio.h>
 #include <X11/Xlib.h>
 
 #include "../util.h"
@@ -11,7 +11,7 @@ keyboard_indicators(void)
        XKeyboardState state;
 
        if (dpy == NULL) {
-               warnx("XOpenDisplay failed");
+               fprintf(stderr, "Cannot open display");
                return NULL;
        }
        XGetKeyboardControl(dpy, &state);
index d6f6bd4bfa04b85cc0687334a55be2d58c08ba8c..5e1571d11c73d160b4aee2e03e1a988bf2d6a760 100644 (file)
@@ -1,5 +1,5 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
+#include <stdio.h>
 #include <stdlib.h>
 
 #include "../util.h"
@@ -10,7 +10,7 @@ load_avg(const char *fmt)
        double avgs[3];
 
        if (getloadavg(avgs, 3) < 0) {
-               warnx("Failed to get the load avg");
+               fprintf(stderr, "Failed to get the load avg");
                return NULL;
        }
 
index a8a389451b9cef5c3ecbccf815738c1e4ecf2395..c471400d8a3aff3ae58e2e8459ae9b6f8be49c2a 100644 (file)
@@ -1,6 +1,5 @@
 /* See LICENSE file for copyright and license details. */
 #include <dirent.h>
-#include <err.h>
 #include <stdio.h>
 #include <string.h>
 
@@ -14,7 +13,7 @@ num_files(const char *dir)
        int num = 0;
 
        if ((fd = opendir(dir)) == NULL) {
-               warn("Failed to get number of files in directory %s", dir);
+               fprintf(stderr, "Failed to get number of files in directory %s", dir);
                return NULL;
        }
 
index 99f54ea7b850021d515c7776457b5fb35ae5897e..4a6389620fdd8ca8169e0bc03f92b74f1e0f669b 100644 (file)
@@ -1,5 +1,4 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
 #include <stdio.h>
 #include <string.h>
 
@@ -13,7 +12,7 @@ run_command(const char *cmd)
 
        fp = popen(cmd, "r");
        if (fp == NULL) {
-               warn("Failed to get command output for %s", cmd);
+               fprintf(stderr, "Failed to get command output for %s", cmd);
                return NULL;
        }
        p = fgets(buf, sizeof(buf) - 1, fp);
index 0aad07491993d499b98d37c7bdb3c1cc115f509b..b82ff46b492553539458e295c9b3eadb85591241 100644 (file)
@@ -1,6 +1,5 @@
 /* See LICENSE file for copyright and license details. */
 #if defined(__linux__)
-#include <err.h>
 #include <stdio.h>
 #include <string.h>
 
@@ -16,12 +15,12 @@ swap_free(void)
 
        fp = fopen("/proc/meminfo", "r");
        if (fp == NULL) {
-               warn("Failed to open file /proc/meminfo");
+               fprintf(stderr, "Failed to open file /proc/meminfo");
                return NULL;
        }
 
        if ((bytes_read = fread(buf, sizeof(char), sizeof(buf) - 1, fp)) == 0) {
-               warn("swap_free: read error");
+               fprintf(stderr, "swap_free: read error");
                fclose(fp);
                return NULL;
        }
@@ -48,12 +47,12 @@ swap_perc(void)
 
        fp = fopen("/proc/meminfo", "r");
        if (fp == NULL) {
-               warn("Failed to open file /proc/meminfo");
+               fprintf(stderr, "Failed to open file /proc/meminfo");
                return NULL;
        }
 
        if ((bytes_read = fread(buf, sizeof(char), sizeof(buf) - 1, fp)) == 0) {
-               warn("swap_perc: read error");
+               fprintf(stderr, "swap_perc: read error");
                fclose(fp);
                return NULL;
        }
@@ -84,11 +83,11 @@ swap_total(void)
 
        fp = fopen("/proc/meminfo", "r");
        if (fp == NULL) {
-               warn("Failed to open file /proc/meminfo");
+               fprintf(stderr, "Failed to open file /proc/meminfo");
                return NULL;
        }
        if ((bytes_read = fread(buf, sizeof(char), sizeof(buf) - 1, fp)) == 0) {
-               warn("swap_total: read error");
+               fprintf(stderr, "swap_total: read error");
                fclose(fp);
                return NULL;
        }
@@ -111,11 +110,11 @@ swap_used(void)
 
        fp = fopen("/proc/meminfo", "r");
        if (fp == NULL) {
-               warn("Failed to open file /proc/meminfo");
+               fprintf(stderr, "Failed to open file /proc/meminfo");
                return NULL;
        }
        if ((bytes_read = fread(buf, sizeof(char), sizeof(buf) - 1, fp)) == 0) {
-               warn("swap_used: read error");
+               fprintf(stderr, "swap_used: read error");
                fclose(fp);
                return NULL;
        }
index 8dcb86a3f8f91c409a2d30be367ea265bafdd52b..ffbd945b8b796b4d60c27c6d7af1a48ec2aff920 100644 (file)
@@ -1,6 +1,6 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
 #include <pwd.h>
+#include <stdio.h>
 #include <sys/types.h>
 #include <unistd.h>
 
@@ -18,7 +18,7 @@ username(void)
        struct passwd *pw = getpwuid(geteuid());
 
        if (pw == NULL) {
-               warn("Failed to get username");
+               fprintf(stderr, "Failed to get username");
                return NULL;
        }
 
index 22fa02fde4f03eaaafb9df98960c899b515f2290..ec653a234c35b0abd5297efee836b87e259c93a6 100644 (file)
@@ -1,6 +1,5 @@
 /* See LICENSE file for copyright and license details. */
 #if defined(__linux__)
-#include <err.h>
 #include <fcntl.h>
 #include <sys/soundcard.h>
 #include <sys/ioctl.h>
@@ -19,19 +18,19 @@ vol_perc(const char *card)
 
        afd = open(card, O_RDONLY | O_NONBLOCK);
        if (afd == -1) {
-               warn("Cannot open %s", card);
+               fprintf(stderr, "Cannot open %s", card);
                return NULL;
        }
 
        if (ioctl(afd, SOUND_MIXER_READ_DEVMASK, &devmask) == -1) {
-               warn("Cannot get volume for %s", card);
+               fprintf(stderr, "Cannot get volume for %s", card);
                close(afd);
                return NULL;
        }
        for (i = 0; i < LEN(vnames); i++) {
                if (devmask & (1 << i) && !strcmp("vol", vnames[i])) {
                        if (ioctl(afd, MIXER_READ(i), &v) == -1) {
-                               warn("vol_perc: ioctl");
+                               fprintf(stderr, "vol_perc: ioctl");
                                close(afd);
                                return NULL;
                        }
index a02c277fe6eddf2070e209c69ca29964881b1286..500332eb6070c25bfd67c7f32cae139963d1ef67 100644 (file)
@@ -1,6 +1,5 @@
 /* See LICENSE file for copyright and license details. */
 #if defined(__linux__)
-#include <err.h>
 #include <ifaddrs.h>
 #include <linux/wireless.h>
 #include <sys/socket.h>
@@ -26,7 +25,7 @@ wifi_perc(const char *iface)
        snprintf(path, sizeof(path), "%s%s%s", "/sys/class/net/", iface, "/operstate");
        fp = fopen(path, "r");
        if (fp == NULL) {
-               warn("Failed to open file %s", path);
+               fprintf(stderr, "Failed to open file %s", path);
                return NULL;
        }
        p = fgets(status, 5, fp);
@@ -37,7 +36,7 @@ wifi_perc(const char *iface)
 
        fp = fopen("/proc/net/wireless", "r");
        if (fp == NULL) {
-               warn("Failed to open file /proc/net/wireless");
+               fprintf(stderr, "Failed to open file /proc/net/wireless");
                return NULL;
        }
 
@@ -72,12 +71,12 @@ wifi_essid(const char *iface)
        snprintf(wreq.ifr_name, sizeof(wreq.ifr_name), "%s", iface);
 
        if (sockfd == -1) {
-               warn("Failed to get ESSID for interface %s", iface);
+               fprintf(stderr, "Failed to get ESSID for interface %s", iface);
                return NULL;
        }
        wreq.u.essid.pointer = id;
        if (ioctl(sockfd,SIOCGIWESSID, &wreq) == -1) {
-               warn("Failed to get ESSID for interface %s", iface);
+               fprintf(stderr, "Failed to get ESSID for interface %s", iface);
                close(sockfd);
                return NULL;
        }
diff --git a/util.c b/util.c
index 67b692a6df8870994e653aa7ff91620e585e6993..cd3524b2c4ed3df9c2317b5e97f1fe90ca06f226 100644 (file)
--- a/util.c
+++ b/util.c
@@ -1,6 +1,4 @@
 /* See LICENSE file for copyright and license details. */
-#include <err.h>
-#include <errno.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <string.h>
@@ -31,7 +29,7 @@ pscanf(const char *path, const char *fmt, ...)
        int n;
 
        if (!(fp = fopen(path, "r"))) {
-               warn("fopen %s: %s\n", path, strerror(errno));
+               fprintf(stderr, "fopen for %s failed", path);
                return -1;
        }
        va_start(ap, fmt);