]> git.ipfire.org Git - thirdparty/libarchive.git/commitdiff
Fix build failure on Windows.
authorMichihiro NAKAJIMA <ggcueroad@gmail.com>
Sun, 12 Oct 2014 05:04:25 +0000 (14:04 +0900)
committerMichihiro NAKAJIMA <ggcueroad@gmail.com>
Sun, 12 Oct 2014 05:10:45 +0000 (14:10 +0900)
 - Visual Studio does not provide unistd.h
 - Visual Studio does not provide getopt
 - Increase portability

Makefile.am
cat/CMakeLists.txt
cat/bsdcat.c
cat/bsdcat.h
cat/bsdcat_platform.h [new file with mode: 0644]
cat/cmdline.c [new file with mode: 0644]

index e7f7292ae450d05eacfba8a8d0d2a663ea6ac600..fbf0a475319b9052bd08447a1481cd35960521b9 100644 (file)
@@ -1127,7 +1127,9 @@ bsdcpio_test_EXTRA_DIST= \
 
 bsdcat_SOURCES= \
                cat/bsdcat.c \
-               cat/bsdcat.h
+               cat/bsdcat.h \
+               cat/bsdcat_platform.h \
+               cat/cmdline.h
 
 if INC_WINDOWS_FILES
 bsdcat_SOURCES+=
index 3eaf78fa865bacbfb478e4a2e54443d259c7082e..48bf0a63d060ebd87c388ee6936462eacffe5976 100644 (file)
@@ -8,6 +8,8 @@ IF(ENABLE_CAT)
   SET(bsdcat_SOURCES
     bsdcat.c
     bsdcat.h
+    bsdcat_platform.h
+    cmdline.c
     ../libarchive_fe/err.c
     ../libarchive_fe/err.h
     ../libarchive_fe/lafe_platform.h
index 8756a8b8868ac46cd0d65c286cd4a2bd3b29c3b0..b06ee97b239f6afc891ac169991072bfd554356e 100644 (file)
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include "bsdcat_platform.h"
+__FBSDID("$FreeBSD$");
+
 #include <stdio.h>
+#ifdef HAVE_STDLIB_H
 #include <stdlib.h>
+#endif
+#ifdef HAVE_UNISTD_H
 #include <unistd.h>
+#endif
+#ifdef HAVE_STRING_H
 #include <string.h>
+#endif
 
 #include "bsdcat.h"
 #include "err.h"
@@ -98,33 +107,37 @@ bsdcat_read_to_stdout(char* filename)
 int
 main(int argc, char **argv)
 {
+       struct bsdcat *bsdcat, bsdcat_storage;
        int c;
 
+       bsdcat = &bsdcat_storage;
+       memset(bsdcat, 0, sizeof(*bsdcat));
+
        lafe_setprogname(*argv, "bsdcat");
 
-       while ((c = getopt(argc, argv, "h-")) != -1) {
+       bsdcat->argv = argv;
+       bsdcat->argc = argc;
+
+       while ((c = bsdcat_getopt(bsdcat)) != -1) {
                switch (c) {
-                       case '-':
-                               if (strcmp(argv[optind], "--version") == 0)
-                                       version();
-                               if (strcmp(argv[optind], "--help") != 0) {
-                                       lafe_warnc(0, "invalid option -- '%s'", argv[optind]);
-                                       usage(stderr, 1);
-                               }
-                       case 'h':
-                               usage(stdout, 0);
-                       default:
-                               usage(stderr, 1);
+               case 'h':
+                       usage(stdout, 0);
+                       break;
+               case OPTION_VERSION:
+                       version();
+                       break;
+               default:
+                       usage(stderr, 1);
                }
        }
 
        bsdcat_next();
-       if (optind >= argc) {
+       if (*bsdcat->argv == NULL) {
                bsdcat_current_path = "<stdin>";
                bsdcat_read_to_stdout(NULL);
        } else
-               while (optind < argc) {
-                       bsdcat_current_path = argv[optind++];
+               while (*bsdcat->argv) {
+                       bsdcat_current_path = *bsdcat->argv++;
                        bsdcat_read_to_stdout(bsdcat_current_path);
                        bsdcat_next();
                }
index 227e01acea7768379726aa05ffe6f741d3fa65a8..ca603d3d6f890d8598b8f7501032f2d80d14a8f1 100644 (file)
 #include <archive.h>
 #include <archive_entry.h>
 
+struct bsdcat {
+       /* Option parser state */
+       int               getopt_state;
+       char             *getopt_word;
+
+       /* Miscellaneous state information */
+       int               argc;
+       char            **argv;
+       const char       *argument;
+};
+
+enum {
+       OPTION_VERSION
+};
+
+int bsdcat_getopt(struct bsdcat *);
 void usage(FILE *stream, int eval);
 void bsdcat_next(void);
 void bsdcat_print_error(void);
diff --git a/cat/bsdcat_platform.h b/cat/bsdcat_platform.h
new file mode 100644 (file)
index 0000000..10b7113
--- /dev/null
@@ -0,0 +1,75 @@
+/*-
+ * Copyright (c) 2003-2007 Tim Kientzle
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR(S) ``AS IS'' AND ANY EXPRESS OR
+ * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+ * IN NO EVENT SHALL THE AUTHOR(S) BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * $FreeBSD: src/usr.bin/tar/bsdtar_platform.h,v 1.26 2008/12/06 07:37:14 kientzle Exp $
+ */
+
+/*
+ * This header is the first thing included in any of the bsdtar
+ * source files.  As far as possible, platform-specific issues should
+ * be dealt with here and not within individual source files.
+ */
+
+#ifndef BSDCAT_PLATFORM_H_INCLUDED
+#define        BSDCAT_PLATFORM_H_INCLUDED
+
+#if defined(PLATFORM_CONFIG_H)
+/* Use hand-built config.h in environments that need it. */
+#include PLATFORM_CONFIG_H
+#else
+/* Not having a config.h of some sort is a serious problem. */
+#include "config.h"
+#endif
+
+/* Get a real definition for __FBSDID if we can */
+#if HAVE_SYS_CDEFS_H
+#include <sys/cdefs.h>
+#endif
+
+/* If not, define it so as to avoid dangling semicolons. */
+#ifndef __FBSDID
+#define        __FBSDID(a)     struct _undefined_hack
+#endif
+
+#ifdef HAVE_LIBARCHIVE
+/* If we're using the platform libarchive, include system headers. */
+#include <archive.h>
+#include <archive_entry.h>
+#else
+/* Otherwise, include user headers. */
+#include "archive.h"
+#include "archive_entry.h"
+#endif
+
+/* How to mark functions that don't return. */
+/* This facilitates use of some newer static code analysis tools. */
+#undef __LA_DEAD
+#if defined(__GNUC__) && (__GNUC__ > 2 || \
+                         (__GNUC__ == 2 && __GNUC_MINOR__ >= 5))
+#define        __LA_DEAD       __attribute__((__noreturn__))
+#else
+#define        __LA_DEAD
+#endif
+
+#endif /* !BSDCAT_PLATFORM_H_INCLUDED */
diff --git a/cat/cmdline.c b/cat/cmdline.c
new file mode 100644 (file)
index 0000000..cae19be
--- /dev/null
@@ -0,0 +1,283 @@
+/*-
+ * Copyright (c) 2003-2008 Tim Kientzle
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR(S) ``AS IS'' AND ANY EXPRESS OR
+ * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+ * IN NO EVENT SHALL THE AUTHOR(S) BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+/*
+ * Command line parser for tar.
+ */
+
+#include "bsdcat_platform.h"
+__FBSDID("$FreeBSD$");
+
+#ifdef HAVE_ERRNO_H
+#include <errno.h>
+#endif
+#ifdef HAVE_STDLIB_H
+#include <stdlib.h>
+#endif
+#ifdef HAVE_STRING_H
+#include <string.h>
+#endif
+
+#include "bsdcat.h"
+#include "err.h"
+
+/*
+ * Short options for tar.  Please keep this sorted.
+ */
+static const char *short_options = "h";
+
+/*
+ * Long options for tar.  Please keep this list sorted.
+ *
+ * The symbolic names for options that lack a short equivalent are
+ * defined in bsdcat.h.  Also note that so far I've found no need
+ * to support optional arguments to long options.  That would be
+ * a small change to the code below.
+ */
+
+static const struct bsdcat_option {
+       const char *name;
+       int required;      /* 1 if this option requires an argument. */
+       int equivalent;    /* Equivalent short option. */
+} tar_longopts[] = {
+       { "help",                 0, 'h' },
+       { "version",              0, OPTION_VERSION },
+       { NULL, 0, 0 }
+};
+
+/*
+ * This getopt implementation has two key features that common
+ * getopt_long() implementations lack.  Apart from those, it's a
+ * straightforward option parser, considerably simplified by not
+ * needing to support the wealth of exotic getopt_long() features.  It
+ * has, of course, been shamelessly tailored for bsdcat.  (If you're
+ * looking for a generic getopt_long() implementation for your
+ * project, I recommend Gregory Pietsch's public domain getopt_long()
+ * implementation.)  The two additional features are:
+ *
+ * Old-style tar arguments: The original tar implementation treated
+ * the first argument word as a list of single-character option
+ * letters.  All arguments follow as separate words.  For example,
+ *    tar xbf 32 /dev/tape
+ * Here, the "xbf" is three option letters, "32" is the argument for
+ * "b" and "/dev/tape" is the argument for "f".  We support this usage
+ * if the first command-line argument does not begin with '-'.  We
+ * also allow regular short and long options to follow, e.g.,
+ *    tar xbf 32 /dev/tape -P --format=pax
+ *
+ * -W long options: There's an obscure GNU convention (only rarely
+ * supported even there) that allows "-W option=argument" as an
+ * alternative way to support long options.  This was supported in
+ * early bsdcat as a way to access long options on platforms that did
+ * not support getopt_long() and is preserved here for backwards
+ * compatibility.  (Of course, if I'd started with a custom
+ * command-line parser from the beginning, I would have had normal
+ * long option support on every platform so that hack wouldn't have
+ * been necessary.  Oh, well.  Some mistakes you just have to live
+ * with.)
+ *
+ * TODO: We should be able to use this to pull files and intermingled
+ * options (such as -C) from the command line in write mode.  That
+ * will require a little rethinking of the argument handling in
+ * bsdcat.c.
+ *
+ * TODO: If we want to support arbitrary command-line options from -T
+ * input (as GNU tar does), we may need to extend this to handle option
+ * words from sources other than argv/argc.  I'm not really sure if I
+ * like that feature of GNU tar, so it's certainly not a priority.
+ */
+
+int
+bsdcat_getopt(struct bsdcat *bsdcat)
+{
+       enum { state_start = 0, state_old_tar, state_next_word,
+              state_short, state_long };
+
+       const struct bsdcat_option *popt, *match = NULL, *match2 = NULL;
+       const char *p, *long_prefix = "--";
+       size_t optlength;
+       int opt = '?';
+       int required = 0;
+
+       bsdcat->argument = NULL;
+
+       /* First time through, initialize everything. */
+       if (bsdcat->getopt_state == state_start) {
+               /* Skip program name. */
+               ++bsdcat->argv;
+               --bsdcat->argc;
+               if (*bsdcat->argv == NULL)
+                       return (-1);
+               /* Decide between "new style" and "old style" arguments. */
+               bsdcat->getopt_state = state_next_word;
+       }
+
+       /*
+        * We're ready to look at the next word in argv.
+        */
+       if (bsdcat->getopt_state == state_next_word) {
+               /* No more arguments, so no more options. */
+               if (bsdcat->argv[0] == NULL)
+                       return (-1);
+               /* Doesn't start with '-', so no more options. */
+               if (bsdcat->argv[0][0] != '-')
+                       return (-1);
+               /* "--" marks end of options; consume it and return. */
+               if (strcmp(bsdcat->argv[0], "--") == 0) {
+                       ++bsdcat->argv;
+                       --bsdcat->argc;
+                       return (-1);
+               }
+               /* Get next word for parsing. */
+               bsdcat->getopt_word = *bsdcat->argv++;
+               --bsdcat->argc;
+               if (bsdcat->getopt_word[1] == '-') {
+                       /* Set up long option parser. */
+                       bsdcat->getopt_state = state_long;
+                       bsdcat->getopt_word += 2; /* Skip leading '--' */
+               } else {
+                       /* Set up short option parser. */
+                       bsdcat->getopt_state = state_short;
+                       ++bsdcat->getopt_word;  /* Skip leading '-' */
+               }
+       }
+
+       /*
+        * We're parsing a group of POSIX-style single-character options.
+        */
+       if (bsdcat->getopt_state == state_short) {
+               /* Peel next option off of a group of short options. */
+               opt = *bsdcat->getopt_word++;
+               if (opt == '\0') {
+                       /* End of this group; recurse to get next option. */
+                       bsdcat->getopt_state = state_next_word;
+                       return bsdcat_getopt(bsdcat);
+               }
+
+               /* Does this option take an argument? */
+               p = strchr(short_options, opt);
+               if (p == NULL)
+                       return ('?');
+               if (p[1] == ':')
+                       required = 1;
+
+               /* If it takes an argument, parse that. */
+               if (required) {
+                       /* If arg is run-in, bsdcat->getopt_word already points to it. */
+                       if (bsdcat->getopt_word[0] == '\0') {
+                               /* Otherwise, pick up the next word. */
+                               bsdcat->getopt_word = *bsdcat->argv;
+                               if (bsdcat->getopt_word == NULL) {
+                                       lafe_warnc(0,
+                                           "Option -%c requires an argument",
+                                           opt);
+                                       return ('?');
+                               }
+                               ++bsdcat->argv;
+                               --bsdcat->argc;
+                       }
+                       if (opt == 'W') {
+                               bsdcat->getopt_state = state_long;
+                               long_prefix = "-W "; /* For clearer errors. */
+                       } else {
+                               bsdcat->getopt_state = state_next_word;
+                               bsdcat->argument = bsdcat->getopt_word;
+                       }
+               }
+       }
+
+       /* We're reading a long option, including -W long=arg convention. */
+       if (bsdcat->getopt_state == state_long) {
+               /* After this long option, we'll be starting a new word. */
+               bsdcat->getopt_state = state_next_word;
+
+               /* Option name ends at '=' if there is one. */
+               p = strchr(bsdcat->getopt_word, '=');
+               if (p != NULL) {
+                       optlength = (size_t)(p - bsdcat->getopt_word);
+                       bsdcat->argument = (char *)(uintptr_t)(p + 1);
+               } else {
+                       optlength = strlen(bsdcat->getopt_word);
+               }
+
+               /* Search the table for an unambiguous match. */
+               for (popt = tar_longopts; popt->name != NULL; popt++) {
+                       /* Short-circuit if first chars don't match. */
+                       if (popt->name[0] != bsdcat->getopt_word[0])
+                               continue;
+                       /* If option is a prefix of name in table, record it.*/
+                       if (strncmp(bsdcat->getopt_word, popt->name, optlength) == 0) {
+                               match2 = match; /* Record up to two matches. */
+                               match = popt;
+                               /* If it's an exact match, we're done. */
+                               if (strlen(popt->name) == optlength) {
+                                       match2 = NULL; /* Forget the others. */
+                                       break;
+                               }
+                       }
+               }
+
+               /* Fail if there wasn't a unique match. */
+               if (match == NULL) {
+                       lafe_warnc(0,
+                           "Option %s%s is not supported",
+                           long_prefix, bsdcat->getopt_word);
+                       return ('?');
+               }
+               if (match2 != NULL) {
+                       lafe_warnc(0,
+                           "Ambiguous option %s%s (matches --%s and --%s)",
+                           long_prefix, bsdcat->getopt_word, match->name, match2->name);
+                       return ('?');
+               }
+
+               /* We've found a unique match; does it need an argument? */
+               if (match->required) {
+                       /* Argument required: get next word if necessary. */
+                       if (bsdcat->argument == NULL) {
+                               bsdcat->argument = *bsdcat->argv;
+                               if (bsdcat->argument == NULL) {
+                                       lafe_warnc(0,
+                                           "Option %s%s requires an argument",
+                                           long_prefix, match->name);
+                                       return ('?');
+                               }
+                               ++bsdcat->argv;
+                               --bsdcat->argc;
+                       }
+               } else {
+                       /* Argument forbidden: fail if there is one. */
+                       if (bsdcat->argument != NULL) {
+                               lafe_warnc(0,
+                                   "Option %s%s does not allow an argument",
+                                   long_prefix, match->name);
+                               return ('?');
+                       }
+               }
+               return (match->equivalent);
+       }
+
+       return (opt);
+}