* configure.in: Make --without-x work.
[deliverable/binutils-gdb.git] / binutils / ar.c
index 7141f42a2e849ab8b217af3d481456dd3d25fdad..ea1b456d9e56b15e3d8979a70e75a6096fd0f394 100644 (file)
@@ -1,29 +1,31 @@
 /* ar.c - Archive modify and extract.
-   Copyright 1991, 92, 93, 94, 95, 96, 97, 1998 Free Software Foundation, Inc.
+   Copyright 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
+   2001, 2002
+   Free Software Foundation, Inc.
 
-This file is part of GNU Binutils.
+   This file is part of GNU Binutils.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 \f
 /*
    Bugs: should use getopt the way tar does (complete w/optional -) and
    should have long options too. GNU ar used to check file against filesystem
    in quick_update and replace operations (would check mtime). Doesn't warn
    when name truncated. No way to specify pos_end. Error messages should be
-   more consistant.
-*/
+   more consistant.  */
+
 #include "bfd.h"
 #include "libiberty.h"
 #include "progress.h"
@@ -31,22 +33,22 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "aout/ar.h"
 #include "libbfd.h"
 #include "arsup.h"
+#include "filenames.h"
+#include "binemul.h"
 #include <sys/stat.h>
 
-#ifdef HAVE_GOOD_UTIME_H
-#include <utime.h>
-#else /* ! HAVE_GOOD_UTIME_H */
-#ifdef HAVE_UTIMES
-#include <sys/time.h>
-#endif /* HAVE_UTIMES */
-#endif /* ! HAVE_GOOD_UTIME_H */
-
 #ifdef __GO32___
 #define EXT_NAME_LEN 3         /* bufflen of addition to name if it's MS-DOS */
 #else
 #define EXT_NAME_LEN 6         /* ditto for *NIX */
 #endif
 
+/* We need to open files in binary modes on system where that makes a
+   difference.  */
+#ifndef O_BINARY
+#define O_BINARY 0
+#endif
+
 #define BUFSIZE 8192
 
 /* Kludge declaration from BFD!  This is ugly!  FIXME!  XXX */
@@ -148,9 +150,19 @@ enum pos
 static bfd **
 get_pos_bfd PARAMS ((bfd **, enum pos, const char *));
 
+/* For extract/delete only.  If COUNTED_NAME_MODE is true, we only
+   extract the COUNTED_NAME_COUNTER instance of that name.  */
+static boolean counted_name_mode = 0;
+static int counted_name_counter = 0;
+
 /* Whether to truncate names of files stored in the archive.  */
 static boolean ar_truncate = false;
 
+/* Whether to use a full file name match when searching an archive.
+   This is convenient for archives created by the Microsoft lib
+   program.  */
+static boolean full_pathname = false;
+
 int interactive = 0;
 
 static void
@@ -172,6 +184,7 @@ map_over_members (arch, function, files, count)
      int count;
 {
   bfd *head;
+  int match_count;
 
   if (count == 0)
     {
@@ -182,6 +195,7 @@ map_over_members (arch, function, files, count)
        }
       return;
     }
+
   /* This may appear to be a baroque way of accomplishing what we want.
      However we have to iterate over the filenames in order to notice where
      a filename is requested but does not exist in the archive.  Ditto
@@ -192,6 +206,7 @@ map_over_members (arch, function, files, count)
     {
       boolean found = false;
 
+      match_count = 0;
       for (head = arch->next; head; head = head->next)
        {
          PROGRESS (1);
@@ -203,13 +218,23 @@ map_over_members (arch, function, files, count)
              bfd_stat_arch_elt (head, &buf);
            }
          if ((head->filename != NULL) &&
-             (!strcmp (*files, head->filename)))
+             (!FILENAME_CMP (normalize (*files, arch), head->filename)))
            {
+             ++match_count;
+             if (counted_name_mode
+                 && match_count != counted_name_counter)
+               {
+                 /* Counting, and didn't match on count; go on to the
+                     next one.  */
+                 continue;
+               }
+
              found = true;
              function (head);
            }
        }
       if (!found)
+       /* xgettext:c-format */
        fprintf (stderr, _("no entry %s in archive\n"), *files);
     }
 }
@@ -223,19 +248,53 @@ usage (help)
   FILE *s;
 
   s = help ? stdout : stderr;
+
   if (! is_ranlib)
-    fprintf (s, _("\
-Usage: %s [-]{dmpqrtx}[abcilosSuvV] [member-name] archive-file file...\n\
-       %s -M [<mri-script]\n"),
-            program_name, program_name);
+    {
+      /* xgettext:c-format */
+      fprintf (s, _("Usage: %s [emulation options] [-]{dmpqrstx}[abcfilNoPsSuvV] [member-name] [count] archive-file file...\n"),
+              program_name);
+      /* xgettext:c-format */
+      fprintf (s, _("       %s -M [<mri-script]\n"), program_name);
+      fprintf (s, _(" commands:\n"));
+      fprintf (s, _("  d            - delete file(s) from the archive\n"));
+      fprintf (s, _("  m[ab]        - move file(s) in the archive\n"));
+      fprintf (s, _("  p            - print file(s) found in the archive\n"));
+      fprintf (s, _("  q[f]         - quick append file(s) to the archive\n"));
+      fprintf (s, _("  r[ab][f][u]  - replace existing or insert new file(s) into the archive\n"));
+      fprintf (s, _("  t            - display contents of archive\n"));
+      fprintf (s, _("  x[o]         - extract file(s) from the archive\n"));
+      fprintf (s, _(" command specific modifiers:\n"));
+      fprintf (s, _("  [a]          - put file(s) after [member-name]\n"));
+      fprintf (s, _("  [b]          - put file(s) before [member-name] (same as [i])\n"));
+      fprintf (s, _("  [N]          - use instance [count] of name\n"));
+      fprintf (s, _("  [f]          - truncate inserted file names\n"));
+      fprintf (s, _("  [P]          - use full path names when matching\n"));
+      fprintf (s, _("  [o]          - preserve original dates\n"));
+      fprintf (s, _("  [u]          - only replace files that are newer than current archive contents\n"));
+      fprintf (s, _(" generic modifiers:\n"));
+      fprintf (s, _("  [c]          - do not warn if the library had to be created\n"));
+      fprintf (s, _("  [s]          - create an archive index (cf. ranlib)\n"));
+      fprintf (s, _("  [S]          - do not build a symbol table\n"));
+      fprintf (s, _("  [v]          - be verbose\n"));
+      fprintf (s, _("  [V]          - display the version number\n"));
+
+      ar_emul_usage (s);
+    }
   else
-    fprintf (s, _("\
-Usage: %s [-vV] archive\n"), program_name);
+    {
+      /* xgettext:c-format */
+      fprintf (s, _("Usage: %s [options] archive\n"), program_name);
+      fprintf (s, _(" Generate an index to speed access to archives\n"));
+      fprintf (s, _(" The options are:\n\
+  -h --help                    Print this help message\n\
+  -V --version                 Print version information\n"));
+    }
 
   list_supported_targets (program_name, stderr);
 
   if (help)
-    fprintf (s, _("Report bugs to bug-gnu-utils@gnu.org\n"));
+    fprintf (s, _("Report bugs to %s\n"), REPORT_BUGS_TO);
 
   xexit (help ? 0 : 1);
 }
@@ -250,7 +309,20 @@ normalize (file, abfd)
 {
   const char *filename;
 
+  if (full_pathname)
+    return file;
+
   filename = strrchr (file, '/');
+#ifdef HAVE_DOS_BASED_FILE_SYSTEM
+  {
+    /* We could have foo/bar\\baz, or foo\\bar, or d:bar.  */
+    char *bslash = strrchr (file, '\\');
+    if (filename == NULL || (bslash != NULL && bslash > filename))
+      filename = bslash;
+    if (filename == NULL && file[0] != '\0' && file[1] == ':')
+      filename = file + 1;
+  }
+#endif
   if (filename != (char *) NULL)
     filename++;
   else
@@ -274,7 +346,7 @@ normalize (file, abfd)
 
 /* Remove any output file.  This is only called via xatexit.  */
 
-static char *output_filename = NULL;
+static const char *output_filename = NULL;
 static FILE *output_file = NULL;
 static bfd *output_bfd = NULL;
 
@@ -294,6 +366,8 @@ remove_output ()
 /* The option parsing should be in its own function.
    It will be when I have getopt working.  */
 
+int main PARAMS ((int, char **));
+
 int
 main (argc, argv)
      int argc;
@@ -308,10 +382,17 @@ main (argc, argv)
     } operation = none;
   int arg_index;
   char **files;
+  int file_count;
   char *inarch_filename;
   int show_version;
+  int i;
 
+#if defined (HAVE_SETLOCALE) && defined (HAVE_LC_MESSAGES)
   setlocale (LC_MESSAGES, "");
+#endif
+#if defined (HAVE_SETLOCALE)
+  setlocale (LC_CTYPE, "");
+#endif
   bindtextdomain (PACKAGE, LOCALEDIR);
   textdomain (PACKAGE);
 
@@ -323,12 +404,22 @@ main (argc, argv)
       char *temp;
 
       temp = strrchr (program_name, '/');
+#ifdef HAVE_DOS_BASED_FILE_SYSTEM
+      {
+       /* We could have foo/bar\\baz, or foo\\bar, or d:bar.  */
+       char *bslash = strrchr (program_name, '\\');
+       if (temp == NULL || (bslash != NULL && bslash > temp))
+         temp = bslash;
+       if (temp == NULL && program_name[0] != '\0' && program_name[1] == ':')
+         temp = program_name + 1;
+      }
+#endif
       if (temp == NULL)
        temp = program_name;
       else
        ++temp;
       if (strlen (temp) >= 6
-         && strcmp (temp + strlen (temp) - 6, "ranlib") == 0)
+         && FILENAME_CMP (temp + strlen (temp) - 6, "ranlib") == 0)
        is_ranlib = 1;
       else
        is_ranlib = 0;
@@ -356,11 +447,20 @@ main (argc, argv)
 
   xatexit (remove_output);
 
+  for (i = 1; i < argc; i++)
+    if (! ar_emul_parse_arg (argv[i]))
+      break;
+  argv += (i - 1);
+  argc -= (i - 1);
+
   if (is_ranlib)
     {
       boolean touch = false;
 
-      if (argc < 2 || strcmp (argv[1], "--help") == 0)
+      if (argc < 2
+         || strcmp (argv[1], "--help") == 0
+         || strcmp (argv[1], "-h") == 0
+         || strcmp (argv[1], "-H") == 0)
        usage (0);
       if (strcmp (argv[1], "-V") == 0
          || strcmp (argv[1], "-v") == 0
@@ -473,11 +573,18 @@ main (argc, argv)
        case 'M':
          mri_mode = 1;
          break;
+       case 'N':
+         counted_name_mode = true;
+         break;
        case 'f':
          ar_truncate = true;
          break;
+       case 'P':
+         full_pathname = true;
+         break;
        default:
-         fprintf (stderr, _("%s: illegal option -- %c\n"), program_name, c);
+         /* xgettext:c-format */
+         non_fatal (_("illegal option -- %c"), c);
          usage (0);
        }
     }
@@ -519,9 +626,19 @@ main (argc, argv)
       if (postype != pos_default)
        posname = argv[arg_index++];
 
+      if (counted_name_mode)
+       {
+         if (operation != extract && operation != delete)
+            fatal (_("`N' is only meaningful with the `x' and `d' options."));
+         counted_name_counter = atoi (argv[arg_index++]);
+         if (counted_name_counter <= 0)
+           fatal (_("Value for `N' must be positive."));
+       }
+
       inarch_filename = argv[arg_index++];
 
       files = arg_index < argc ? argv + arg_index : NULL;
+      file_count = argc - arg_index;
 
 #if 0
       /* We don't use do_quick_append any more.  Too many systems
@@ -550,7 +667,7 @@ main (argc, argv)
       if (operation == quick_append)
        {
          /* Note that quick appending to a non-existent archive creates it,
-            even if there are no files to append. */
+            even if there are no files to append.  */
          do_quick_append (inarch_filename, files);
          xexit (0);
        }
@@ -562,38 +679,43 @@ main (argc, argv)
       switch (operation)
        {
        case print_table:
-         map_over_members (arch, print_descr, files, argc - 3);
+         map_over_members (arch, print_descr, files, file_count);
          break;
 
        case print_files:
-         map_over_members (arch, print_contents, files, argc - 3);
+         map_over_members (arch, print_contents, files, file_count);
          break;
 
        case extract:
-         map_over_members (arch, extract_file, files, argc - 3);
+         map_over_members (arch, extract_file, files, file_count);
          break;
 
        case delete:
          if (files != NULL)
            delete_members (arch, files);
+         else
+           output_filename = NULL;
          break;
 
        case move:
          if (files != NULL)
            move_members (arch, files);
+         else
+           output_filename = NULL;
          break;
 
        case replace:
        case quick_append:
          if (files != NULL || write_armap > 0)
            replace_members (arch, files, operation == quick_append);
+         else
+           output_filename = NULL;
          break;
 
          /* Shouldn't happen! */
        default:
-         fprintf (stderr, _("%s: internal error -- this option not implemented\n"),
-                  program_name);
-         xexit (1);
+         /* xgettext:c-format */
+         fatal (_("internal error -- this option not implemented"));
        }
     }
 
@@ -621,12 +743,15 @@ open_inarch (archive_filename, file)
 
   if (stat (archive_filename, &sbuf) != 0)
     {
-#ifndef __GO32__
+#if !defined(__GO32__) || defined(__DJGPP__)
+
+      /* FIXME: I don't understand why this fragment was ifndef'ed
+        away for __GO32__; perhaps it was in the days of DJGPP v1.x.
+        stat() works just fine in v2.x, so I think this should be
+        removed.  For now, I enable it for DJGPP v2. -- EZ.  */
 
 /* KLUDGE ALERT! Temporary fix until I figger why
- * stat() is wrong ... think it's buried in GO32's IDT
- * - Jax
- */
+   stat() is wrong ... think it's buried in GO32's IDT - Jax */
       if (errno != ENOENT)
        bfd_fatal (archive_filename);
 #endif
@@ -660,6 +785,9 @@ open_inarch (archive_filename, file)
          || ! bfd_set_format (arch, bfd_archive)
          || ! bfd_close (arch))
        bfd_fatal (archive_filename);
+
+      /* If we die creating a new archive, don't leave it around.  */
+      output_filename = archive_filename;
     }
 
   arch = bfd_openr (archive_filename, target);
@@ -705,12 +833,14 @@ print_contents (abfd)
   struct stat buf;
   long size;
   if (bfd_stat_arch_elt (abfd, &buf) != 0)
+    /* xgettext:c-format */
     fatal (_("internal stat error on %s"), bfd_get_filename (abfd));
 
   if (verbose)
+    /* xgettext:c-format */
     printf (_("\n<member %s>\n\n"), bfd_get_filename (abfd));
 
-  bfd_seek (abfd, 0, SEEK_SET);
+  bfd_seek (abfd, (file_ptr) 0, SEEK_SET);
 
   size = buf.st_size;
   while (ncopied < size)
@@ -721,9 +851,9 @@ print_contents (abfd)
       if (tocopy > BUFSIZE)
        tocopy = BUFSIZE;
 
-      nread = bfd_read (cbuf, 1, tocopy, abfd);        /* oops -- broke
-                                                          abstraction!  */
+      nread = bfd_bread (cbuf, (bfd_size_type) tocopy, abfd);
       if (nread != tocopy)
+       /* xgettext:c-format */
        fatal (_("%s is not a valid archive"),
               bfd_get_filename (bfd_my_archive (abfd)));
       fwrite (cbuf, 1, nread, stdout);
@@ -749,26 +879,32 @@ extract_file (abfd)
   FILE *ostream;
   char *cbuf = xmalloc (BUFSIZE);
   int nread, tocopy;
-  int ncopied = 0;
+  long ncopied = 0;
   long size;
   struct stat buf;
+
   if (bfd_stat_arch_elt (abfd, &buf) != 0)
+    /* xgettext:c-format */
     fatal (_("internal stat error on %s"), bfd_get_filename (abfd));
   size = buf.st_size;
 
+  if (size < 0)
+    /* xgettext:c-format */
+    fatal (_("stat returns negative size for %s"), bfd_get_filename (abfd));
+
   if (verbose)
     printf ("x - %s\n", bfd_get_filename (abfd));
 
-  bfd_seek (abfd, 0, SEEK_SET);
+  bfd_seek (abfd, (file_ptr) 0, SEEK_SET);
 
-  ostream = 0;
+  ostream = NULL;
   if (size == 0)
     {
       /* Seems like an abstraction violation, eh?  Well it's OK! */
       output_filename = bfd_get_filename (abfd);
 
       ostream = fopen (bfd_get_filename (abfd), FOPEN_WB);
-      if (!ostream)
+      if (ostream == NULL)
        {
          perror (bfd_get_filename (abfd));
          xexit (1);
@@ -783,19 +919,20 @@ extract_file (abfd)
        if (tocopy > BUFSIZE)
          tocopy = BUFSIZE;
 
-       nread = bfd_read (cbuf, 1, tocopy, abfd);
+       nread = bfd_bread (cbuf, (bfd_size_type) tocopy, abfd);
        if (nread != tocopy)
+         /* xgettext:c-format */
          fatal (_("%s is not a valid archive"),
                 bfd_get_filename (bfd_my_archive (abfd)));
 
        /* See comment above; this saves disk arm motion */
-       if (!ostream)
+       if (ostream == NULL)
          {
            /* Seems like an abstraction violation, eh?  Well it's OK! */
            output_filename = bfd_get_filename (abfd);
 
            ostream = fopen (bfd_get_filename (abfd), FOPEN_WB);
-           if (!ostream)
+           if (ostream == NULL)
              {
                perror (bfd_get_filename (abfd));
                xexit (1);
@@ -807,7 +944,8 @@ extract_file (abfd)
        ncopied += tocopy;
       }
 
-  fclose (ostream);
+  if (ostream != NULL)
+    fclose (ostream);
 
   output_file = NULL;
   output_filename = NULL;
@@ -815,29 +953,9 @@ extract_file (abfd)
   chmod (bfd_get_filename (abfd), buf.st_mode);
 
   if (preserve_dates)
-    {
-#ifdef HAVE_GOOD_UTIME_H
-      struct utimbuf tb;
-      tb.actime = buf.st_mtime;
-      tb.modtime = buf.st_mtime;
-      utime (bfd_get_filename (abfd), &tb);    /* FIXME check result */
-#else /* ! HAVE_GOOD_UTIME_H */
-#ifndef HAVE_UTIMES
-      long tb[2];
-      tb[0] = buf.st_mtime;
-      tb[1] = buf.st_mtime;
-      utime (bfd_get_filename (abfd), tb);     /* FIXME check result */
-#else /* HAVE_UTIMES */
-      struct timeval tv[2];
-      tv[0].tv_sec = buf.st_mtime;
-      tv[0].tv_usec = 0;
-      tv[1].tv_sec = buf.st_mtime;
-      tv[1].tv_usec = 0;
-      utimes (bfd_get_filename (abfd), tv);    /* FIXME check result */
-#endif /* HAVE_UTIMES */
-#endif /* ! HAVE_GOOD_UTIME_H */
-    }
-free (cbuf);
+    set_times (bfd_get_filename (abfd), &buf);
+
+  free (cbuf);
 }
 
 #if 0
@@ -863,12 +981,18 @@ do_quick_append (archive_filename, files_to_append)
   if (stat (archive_filename, &sbuf) != 0)
     {
 
-#ifndef __GO32__
+#if !defined(__GO32__) || defined(__DJGPP__)
+
+      /* FIXME: I don't understand why this fragment was ifndef'ed
+        away for __GO32__; perhaps it was in the days of DJGPP v1.x.
+        stat() works just fine in v2.x, so I think this should be
+        removed.  For now, I enable it for DJGPP v2.
+
+        (And yes, I know this is all unused, but somebody, someday,
+        might wish to resurrect this again... -- EZ.  */
 
 /* KLUDGE ALERT! Temporary fix until I figger why
- * stat() is wrong ... think it's buried in GO32's IDT
- * - Jax
- */
+   stat() is wrong ... think it's buried in GO32's IDT - Jax  */
 
       if (errno != ENOENT)
        bfd_fatal (archive_filename);
@@ -892,14 +1016,15 @@ do_quick_append (archive_filename, files_to_append)
   if (newfile == false)
     {
       if (bfd_check_format (temp, bfd_archive) != true)
+       /* xgettext:c-format */
        fatal (_("%s is not an archive"), archive_filename);
     }
   else
     {
       fwrite (ARMAG, 1, SARMAG, ofile);
       if (!silent_create)
-       fprintf (stderr, _("%s: creating %s\n"),
-                program_name, archive_filename);
+       /* xgettext:c-format */
+       non_fatal (_("creating %s"), archive_filename);
     }
 
   if (ar_truncate)
@@ -999,10 +1124,9 @@ write_archive (iarch)
 
   /* We don't care if this fails; we might be creating the archive.  */
   bfd_close (iarch);
-  unlink (old_name);
 
-  if (rename (new_name, old_name) != 0)
-    bfd_fatal (old_name);
+  if (smart_rename (new_name, old_name, 0) != 0)
+    xexit (1);
 }
 
 /* Return a pointer to the pointer to the entry which should be rplacd'd
@@ -1038,7 +1162,7 @@ get_pos_bfd (contents, default_pos, default_posname)
   else
     {
       for (; *after_bfd; after_bfd = &(*after_bfd)->next)
-       if (strcmp ((*after_bfd)->filename, realposname) == 0)
+       if (FILENAME_CMP ((*after_bfd)->filename, realposname) == 0)
          {
            if (realpos == pos_after)
              after_bfd = &(*after_bfd)->next;
@@ -1056,6 +1180,8 @@ delete_members (arch, files_to_delete)
   bfd **current_ptr_ptr;
   boolean found;
   boolean something_changed = false;
+  int match_count;
+
   for (; *files_to_delete != NULL; ++files_to_delete)
     {
       /* In a.out systems, the armap is optional.  It's also called
@@ -1072,27 +1198,38 @@ delete_members (arch, files_to_delete)
        }
 
       found = false;
+      match_count = 0;
       current_ptr_ptr = &(arch->next);
       while (*current_ptr_ptr)
        {
-         if (strcmp (*files_to_delete, (*current_ptr_ptr)->filename) == 0)
+         if (FILENAME_CMP (normalize (*files_to_delete, arch),
+                           (*current_ptr_ptr)->filename) == 0)
            {
-             found = true;
-             something_changed = true;
-             if (verbose)
-               printf ("d - %s\n",
-                       *files_to_delete);
-             *current_ptr_ptr = ((*current_ptr_ptr)->next);
-             goto next_file;
-           }
-         else
-           {
-             current_ptr_ptr = &((*current_ptr_ptr)->next);
+             ++match_count;
+             if (counted_name_mode
+                 && match_count != counted_name_counter)
+               {
+                 /* Counting, and didn't match on count; go on to the
+                     next one.  */
+               }
+             else
+               {
+                 found = true;
+                 something_changed = true;
+                 if (verbose)
+                   printf ("d - %s\n",
+                           *files_to_delete);
+                 *current_ptr_ptr = ((*current_ptr_ptr)->next);
+                 goto next_file;
+               }
            }
+
+         current_ptr_ptr = &((*current_ptr_ptr)->next);
        }
 
       if (verbose && found == false)
        {
+         /* xgettext:c-format */
          printf (_("No member named `%s'\n"), *files_to_delete);
        }
     next_file:
@@ -1100,9 +1237,9 @@ delete_members (arch, files_to_delete)
     }
 
   if (something_changed == true)
-    {
-      write_archive (arch);
-    }
+    write_archive (arch);
+  else
+    output_filename = NULL;
 }
 
 
@@ -1122,8 +1259,8 @@ move_members (arch, files_to_move)
       while (*current_ptr_ptr)
        {
          bfd *current_ptr = *current_ptr_ptr;
-         if (strcmp (normalize (*files_to_move, arch),
-                     current_ptr->filename) == 0)
+         if (FILENAME_CMP (normalize (*files_to_move, arch),
+                           current_ptr->filename) == 0)
            {
              /* Move this file to the end of the list - first cut from
                 where it is.  */
@@ -1144,9 +1281,9 @@ move_members (arch, files_to_move)
 
          current_ptr_ptr = &((*current_ptr_ptr)->next);
        }
-      fprintf (stderr, _("%s: no entry %s in archive %s!\n"),
-              program_name, *files_to_move, arch->filename);
-      xexit (1);
+      /* xgettext:c-format */
+      fatal (_("no entry %s in archive %s!"), *files_to_move, arch->filename);
+
     next_file:;
     }
 
@@ -1165,7 +1302,6 @@ replace_members (arch, files_to_move, quick)
   bfd **after_bfd;             /* New entries go after this one */
   bfd *current;
   bfd **current_ptr;
-  bfd *temp;
 
   while (files_to_move && *files_to_move)
     {
@@ -1178,8 +1314,8 @@ replace_members (arch, files_to_move, quick)
 
              /* For compatibility with existing ar programs, we
                 permit the same file to be added multiple times.  */
-             if (strcmp (normalize (*files_to_move, arch),
-                         normalize (current->filename, arch)) == 0
+             if (FILENAME_CMP (normalize (*files_to_move, arch),
+                               normalize (current->filename, arch)) == 0
                  && current->arelt_data != NULL)
                {
                  if (newer_only)
@@ -1193,7 +1329,9 @@ replace_members (arch, files_to_move, quick)
                          goto next_file;
                        }
                      if (bfd_stat_arch_elt (current, &asbuf) != 0)
-                       fatal (_("internal stat error on %s"), current->filename);
+                       /* xgettext:c-format */
+                       fatal (_("internal stat error on %s"),
+                              current->filename);
 
                      if (fsbuf.st_mtime <= asbuf.st_mtime)
                        goto next_file;
@@ -1201,24 +1339,13 @@ replace_members (arch, files_to_move, quick)
 
                  after_bfd = get_pos_bfd (&arch->next, pos_after,
                                           current->filename);
-                 temp = *after_bfd;
-
-                 *after_bfd = bfd_openr (*files_to_move, NULL);
-                 if (*after_bfd == (bfd *) NULL)
+                 if (ar_emul_replace (after_bfd, *files_to_move,
+                                      verbose))
                    {
-                     bfd_fatal (*files_to_move);
+                     /* Snip out this entry from the chain.  */
+                     *current_ptr = (*current_ptr)->next;
+                     changed = true;
                    }
-                 (*after_bfd)->next = temp;
-
-                 /* snip out this entry from the chain */
-                 *current_ptr = (*current_ptr)->next;
-
-                 if (verbose)
-                   {
-                     printf ("r - %s\n", *files_to_move);
-                   }
-
-                 changed = true;
 
                  goto next_file;
                }
@@ -1227,22 +1354,9 @@ replace_members (arch, files_to_move, quick)
        }
 
       /* Add to the end of the archive.  */
-
       after_bfd = get_pos_bfd (&arch->next, pos_end, NULL);
-      temp = *after_bfd;
-      *after_bfd = bfd_openr (*files_to_move, NULL);
-      if (*after_bfd == (bfd *) NULL)
-       {
-         bfd_fatal (*files_to_move);
-       }
-      if (verbose)
-       {
-         printf ("a - %s\n", *files_to_move);
-       }
-
-      (*after_bfd)->next = temp;
-
-      changed = true;
+      if (ar_emul_append (after_bfd, *files_to_move, verbose))
+       changed = true;
 
     next_file:;
 
@@ -1251,6 +1365,8 @@ replace_members (arch, files_to_move, quick)
 
   if (changed)
     write_archive (arch);
+  else
+    output_filename = NULL;
 }
 
 static void
@@ -1280,7 +1396,7 @@ ranlib_touch (archname)
   bfd *arch;
   char **matching;
 
-  f = open (archname, O_RDWR, 0);
+  f = open (archname, O_RDWR | O_BINARY, 0);
   if (f < 0)
     {
       bfd_set_error (bfd_error_system_call);
@@ -1302,6 +1418,7 @@ ranlib_touch (archname)
     }
 
   if (! bfd_has_map (arch))
+    /* xgettext:c-format */
     fatal (_("%s: no archive map to update"), archname);
 
   bfd_update_armap_timestamp (arch);
This page took 0.03527 seconds and 4 git commands to generate.