[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Translator support for GNU tar (patch 2)
|
From: |
Marco Gerards |
|
Subject: |
Re: Translator support for GNU tar (patch 2) |
|
Date: |
22 Jun 2003 23:43:11 +0200 |
|
User-agent: |
Gnus/5.09 (Gnus v5.9.0) Emacs/21.2 |
And here is my second attempt. I think I've fixed all minor problems
everyone had with my last patch (Thanks Alfred, Marcus and
Wolfgang).
I would also like some comments about this patch. Please tell me what
to change, add, do before sending it to the tar mailinglist.
What I also like to know is how to detect Hurd features with
autoconf. Wolfgang suggested something else than Alfred and now I am
confused what to do.
Thanks,
Marco
diff -urp tar-1.13.25/configure.ac tar-1.13.25_hurd/configure.ac
--- tar-1.13.25/configure.ac 2001-09-26 22:30:53.000000000 +0200
+++ tar-1.13.25_hurd/configure.ac 2003-06-22 23:25:36.000000000 +0200
@@ -1,7 +1,7 @@
# Configure template for GNU tar.
-# Copyright (C) 1991, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001 Free
-# Software Foundation, Inc.
+# Copyright (C) 1991, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001,
+# 2003 Free Software Foundation, Inc.
# 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
@@ -47,7 +47,7 @@ AC_CHECK_HEADERS(fcntl.h limits.h linux/
sys/buf.h sys/device.h sys/gentape.h \
sys/inet.h sys/io/trioctl.h sys/ioccom.h \
sys/mtio.h sys/param.h sys/tprintf.h sys/tape.h sys/time.h sys/timeb.h \
- unistd.h wchar.h wctype.h)
+ unistd.h wchar.h wctype.h hurd.h)
AC_HEADER_SYS_WAIT
Only in tar-1.13.25_hurd: configure.ac~
diff -urp tar-1.13.25/src/common.h tar-1.13.25_hurd/src/common.h
--- tar-1.13.25/src/common.h 2001-09-21 02:00:55.000000000 +0200
+++ tar-1.13.25_hurd/src/common.h 2003-06-22 23:09:07.000000000 +0200
@@ -1,7 +1,7 @@
/* Common declarations for the tar program.
- Copyright 1988, 1992, 1993, 1994, 1996, 1997, 1999, 2000, 2001 Free
- Software Foundation, Inc.
+ Copyright 1988, 1992, 1993, 1994, 1996, 1997, 1999, 2000, 2001,
+ 2003 Free Software Foundation, Inc.
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
@@ -149,6 +149,9 @@ GLOBAL const char *use_compress_program_
/* Boolean value. */
GLOBAL int dereference_option;
+/* Boolean value. */
+GLOBAL int dump_translators_option;
+
/* Patterns that match file names to be excluded. */
GLOBAL struct exclude *excluded;
diff -urp tar-1.13.25/src/compare.c tar-1.13.25_hurd/src/compare.c
--- tar-1.13.25/src/compare.c 2001-09-22 02:47:09.000000000 +0200
+++ tar-1.13.25_hurd/src/compare.c 2003-06-22 23:09:07.000000000 +0200
@@ -1,7 +1,7 @@
/* Diff files from a tar archive.
- Copyright 1988, 1992, 1993, 1994, 1996, 1997, 1999, 2000, 2001 Free
- Software Foundation, Inc.
+ Copyright 1988, 1992, 1993, 1994, 1996, 1997, 1999, 2000, 2001,
+ 2003 Free Software Foundation, Inc.
Written by John Gilmore, on 1987-04-30.
@@ -37,6 +37,13 @@ struct utimbuf
#include <quotearg.h>
+#if HAVE_HURD_H
+# include <hurd.h>
+# if HAVE_ARGZ_H
+# include <argz.h>
+# endif
+#endif
+
#include "common.h"
#include "rmt.h"
@@ -734,6 +741,60 @@ diff_archive (void)
break;
}
+
+#if HAVE_HURD_H
+ case GNUTYPE_TRANS:
+ {
+ size_t len = strlen (current_link_name);
+ size_t translen = len;
+ char *buf = alloca (len + 1);
+ char *transbuf = buf;
+
+ file_t node;
+
+ node = file_name_lookup (current_file_name, O_NOTRANS, 0);
+ if (node == MACH_PORT_NULL)
+ {
+ gettrans_warn (current_file_name);
+ report_difference (0);
+ skip_member ();
+ break;
+ }
+
+ status = file_get_translator (node, &transbuf, &translen);
+ if (status == EINVAL)
+ {
+ mach_port_deallocate (mach_task_self (), node);
+ gettrans_warn (current_file_name);
+ report_difference (0);
+ skip_member ();
+ break;
+ }
+
+ if (status)
+ {
+ mach_port_deallocate (mach_task_self (), node);
+ gettrans_error (current_file_name);
+ report_difference (0);
+ break;
+ }
+ else
+ {
+ argz_stringify (transbuf, translen, 1);
+
+ if (len != translen
+ || strncmp (current_link_name, transbuf, len) != 0)
+ report_difference (_("Translator differs"));
+ }
+
+ if (transbuf != buf)
+ munmap (transbuf, translen);
+
+ mach_port_deallocate (mach_task_self (), node);
+
+ break;
+ }
+#endif
}
}
diff -urp tar-1.13.25/src/create.c tar-1.13.25_hurd/src/create.c
--- tar-1.13.25/src/create.c 2001-08-29 23:21:02.000000000 +0200
+++ tar-1.13.25_hurd/src/create.c 2003-06-22 23:09:07.000000000 +0200
@@ -1,5 +1,7 @@
/* Create a tar archive.
- Copyright 1985,92,93,94,96,97,99,2000, 2001 Free Software Foundation, Inc.
+ Copyright 1985,92,93,94,96,97,99,2000, 2001, 2003
+ Free Software Foundation, Inc.
+
Written by John Gilmore, on 1985-08-25.
This program is free software; you can redistribute it and/or modify it
@@ -38,6 +40,13 @@ struct utimbuf
#include "common.h"
#include <hash.h>
+#if HAVE_HURD_H
+# include <hurd.h>
+# if HAVE_ARGZ_H
+# include <argz.h>
+# endif
+#endif
+
#ifndef MSDOS
extern dev_t ar_dev;
extern ino_t ar_ino;
@@ -351,7 +360,8 @@ write_eot (void)
set_next_block_after (pointer);
}
-/* Write a GNUTYPE_LONGLINK or GNUTYPE_LONGNAME block. */
+/* Write a GNUTYPE_LONGLINK, GNUTYPE_LONGNAME or GNUTYPE_LONGTRANS
+ block. */
/* FIXME: Cross recursion between start_header and write_long! */
@@ -568,7 +578,8 @@ finish_header (union block *header)
if (verbose_option
&& header->header.typeflag != GNUTYPE_LONGLINK
- && header->header.typeflag != GNUTYPE_LONGNAME)
+ && header->header.typeflag != GNUTYPE_LONGNAME
+ && header->header.typeflag != GNUTYPE_LONGTRANS)
{
/* These globals are parameters to print_header, sigh. */
@@ -902,6 +913,7 @@ dump_file (char *p, int top_level, dev_t
char save_typeflag;
time_t original_ctime;
struct utimbuf restore_times;
+ int status;
/* FIXME: `header' might be used uninitialized in this
function. Reported by Bruno Haible. */
@@ -909,6 +921,83 @@ dump_file (char *p, int top_level, dev_t
if (interactive_option && !confirm ("add", p))
return;
+#if HAVE_HURD_H
+ /* Set a translator. There is a better way to descibe the translator
+ (symlink for example) this translator will be overwritten. */
+ if (dump_translators_option)
+ {
+ file_t node;
+ char buf[1024];
+ char *transbuf = buf;
+ size_t translen = sizeof (buf);
+
+ node = file_name_lookup (p, O_NOTRANS, 0);
+ if (node == MACH_PORT_NULL)
+ {
+ open_error (p);
+ return;
+ }
+ else
+ {
+ if (io_stat (node, ¤t_stat) != 0)
+ {
+ mach_port_deallocate (mach_task_self (), node);
+
+ if (ignore_failed_read_option)
+ stat_warn (p);
+ else
+ stat_error (p);
+
+ return;
+ }
+
+ status = file_get_translator (node, &transbuf, &translen);
+
+ mach_port_deallocate (mach_task_self (), node);
+
+ if (status && status != EINVAL)
+ {
+ gettrans_error (p);
+ return;
+ }
+ else if (status != EINVAL)
+ {
+ /* Check if the node is a translator of a type that
+ should be stored as a translator. */
+ if (!strcmp ("/hurd/symlink", transbuf) ||
+ !strcmp ("/hurd/fifo", transbuf))
+ {
+ if (transbuf != buf)
+ munmap (transbuf, translen);
+ }
+ else
+ {
+ /* FIXME: The translator is stored in a argz
+ list. These lists have parameters seperated by
+ 0's. Replace the 0's by 1's so it can se stored
+ by tar. */
+ argz_stringify (transbuf, translen, 1);
+
+ if (NAME_FIELD_SIZE <= translen)
+ write_long (transbuf, GNUTYPE_LONGTRANS);
+
+ header = start_header (p, ¤t_stat);
+ assign_string (¤t_link_name, transbuf);
+ strncpy (header->header.linkname, transbuf, NAME_FIELD_SIZE);
+ header->header.linkname[NAME_FIELD_SIZE - 1] = '\0';
+ header->header.typeflag = GNUTYPE_TRANS;
+ finish_header (header);
+
+ if (transbuf != buf)
+ munmap (transbuf, translen);
+
+ return;
+ }
+ }
+ }
+ }
+#endif
+
if (deref_stat (dereference_option, p, ¤t_stat) != 0)
{
if (ignore_failed_read_option)
@@ -917,7 +1006,7 @@ dump_file (char *p, int top_level, dev_t
stat_error (p);
return;
}
-
+
original_ctime = current_stat.st_ctime;
restore_times.actime = current_stat.st_atime;
restore_times.modtime = current_stat.st_mtime;
diff -urp tar-1.13.25/src/extract.c tar-1.13.25_hurd/src/extract.c
--- tar-1.13.25/src/extract.c 2001-09-24 20:55:17.000000000 +0200
+++ tar-1.13.25_hurd/src/extract.c 2003-06-22 23:09:07.000000000 +0200
@@ -1,7 +1,7 @@
/* Extract files from a tar archive.
Copyright 1988, 1992, 1993, 1994, 1996, 1997, 1998, 1999, 2000,
- 2001 Free Software Foundation, Inc.
+ 2001, 2003 Free Software Foundation, Inc.
Written by John Gilmore, on 1985-11-19.
@@ -22,6 +22,13 @@
#include "system.h"
#include <quotearg.h>
+#if HAVE_HURD_H
+# include <hurd.h>
+# if HAVE_ARGZ_H
+# include <argz.h>
+# endif
+#endif
+
#if HAVE_UTIME_H
# include <utime.h>
#else
@@ -1213,12 +1220,60 @@ extract_archive (void)
case GNUTYPE_LONGNAME:
case GNUTYPE_LONGLINK:
+ case GNUTYPE_LONGTRANS:
ERROR ((0, 0, _("Visible long name error")));
skip_member ();
if (backup_option)
undo_last_backup ();
break;
+#if HAVE_HURD_H
+ case GNUTYPE_TRANS:
+ {
+ file_t node;
+ char *argz = 0;
+ size_t argz_len = 0;
+
+ if (!prepare_to_extract (CURRENT_FILE_NAME))
+ break;
+
+ node = file_name_lookup (CURRENT_FILE_NAME, O_NOTRANS | O_CREAT,
+ 600);
+
+ if (node == MACH_PORT_NULL)
+ {
+ open_error (CURRENT_FILE_NAME);
+ break;
+ }
+
+ set_stat (CURRENT_FILE_NAME, ¤t_stat, 0, 0,
+ ARCHIVED_PERMSTATUS, typeflag);
+
+ status = argz_create_sep (current_link_name, 1, &argz, &argz_len);
+
+ if (status)
+ {
+ settrans_error (CURRENT_FILE_NAME);
+ mach_port_deallocate (mach_task_self (), node);
+ break;
+ }
+
+ status = file_set_translator (node,
+ FS_TRANS_SET | FS_TRANS_EXCL,
+ FS_TRANS_SET | FS_TRANS_EXCL, 0,
+ argz, argz_len,
+ MACH_PORT_NULL, MACH_MSG_TYPE_COPY_SEND);
+
+ free (argz);
+
+ if (status)
+ settrans_error (CURRENT_FILE_NAME);
+
+ mach_port_deallocate (mach_task_self (), node);
+ break;
+ }
+#endif
+
default:
WARN ((0, 0,
_("%s: Unknown file type '%c', extracted as normal file"),
diff -urp tar-1.13.25/src/list.c tar-1.13.25_hurd/src/list.c
--- tar-1.13.25/src/list.c 2001-09-26 22:05:04.000000000 +0200
+++ tar-1.13.25_hurd/src/list.c 2003-06-22 23:09:07.000000000 +0200
@@ -1,7 +1,7 @@
/* List a tar archive, with support routines for reading a tar archive.
Copyright 1988, 1992, 1993, 1994, 1996, 1997, 1998, 1999, 2000,
- 2001 Free Software Foundation, Inc.
+ 2001, 2003 Free Software Foundation, Inc.
Written by John Gilmore, on 1985-08-26.
@@ -326,7 +326,8 @@ read_header (bool raw_extended_headers)
current_stat.st_size = OFF_FROM_HEADER (header->header.size);
if (header->header.typeflag == GNUTYPE_LONGNAME
- || header->header.typeflag == GNUTYPE_LONGLINK)
+ || header->header.typeflag == GNUTYPE_LONGLINK
+ || header->header.typeflag == GNUTYPE_LONGTRANS)
{
if (raw_extended_headers)
return HEADER_SUCCESS_EXTENDED;
@@ -941,6 +942,10 @@ print_header (void)
modes[0] = 'N';
break;
+ case GNUTYPE_TRANS:
+ modes[0] = 'T';
+ break;
+
case GNUTYPE_LONGNAME:
case GNUTYPE_LONGLINK:
ERROR ((0, 0, _("Visible longname error")));
@@ -1074,6 +1079,22 @@ print_header (void)
fprintf (stdlis, _(" link to %s\n"), quotearg (current_link_name));
break;
+ case GNUTYPE_TRANS:
+ {
+ char *s = current_link_name;
+
+ /* Use the seperator ' ' instead of 1 to print the string. */
+ while (*s)
+ {
+ if (*s == 1)
+ *s = ' ';
+ s++;
+ }
+
+ fprintf (stdlis, " => %s\n", quotearg (current_link_name));
+ break;
+ }
+
default:
{
char type_string[2];
diff -urp tar-1.13.25/src/misc.c tar-1.13.25_hurd/src/misc.c
--- tar-1.13.25/src/misc.c 2001-08-27 01:14:26.000000000 +0200
+++ tar-1.13.25_hurd/src/misc.c 2003-06-22 23:09:07.000000000 +0200
@@ -1,7 +1,7 @@
/* Miscellaneous functions, not really specific to GNU tar.
- Copyright 1988, 1992, 1994, 1995, 1996, 1997, 1999, 2000, 2001 Free
- Software Foundation, Inc.
+ Copyright 1988, 1992, 1994, 1995, 1996, 1997, 1999, 2000, 2001,
+ 2003 Free Software Foundation, Inc.
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
@@ -23,6 +23,10 @@
#include <quotearg.h>
#include <save-cwd.h>
+#if HAVE_HURD_H
+# include <hurd.h>
+#endif
+
static void call_arg_fatal PARAMS ((char const *, char const *))
__attribute__ ((noreturn));
@@ -688,6 +692,26 @@ readlink_warn (char const *name)
call_arg_warn ("readlink", name);
}
+#if HAVE_HURD_H
+void
+gettrans_error (char const *name)
+{
+ call_arg_error ("gettrans", name);
+}
+
+void
+gettrans_warn (char const *name)
+{
+ call_arg_warn ("gettrans", name);
+}
+
+void
+settrans_error (char const *name)
+{
+ call_arg_error ("gettrans", name);
+}
+#endif
+
void
savedir_error (char const *name)
{
diff -urp tar-1.13.25/src/tar.c tar-1.13.25_hurd/src/tar.c
--- tar-1.13.25/src/tar.c 2001-09-21 02:11:27.000000000 +0200
+++ tar-1.13.25_hurd/src/tar.c 2003-06-22 23:09:07.000000000 +0200
@@ -1,7 +1,7 @@
/* A tar (tape archiver) program.
- Copyright (C) 1988, 1992, 1993, 1994, 1995, 1996, 1997, 1999, 2000, 2001
- Free Software Foundation, Inc.
+ Copyright (C) 1988, 1992, 1993, 1994, 1995, 1996, 1997, 1999, 2000,
+ 2001, 2003 Free Software Foundation, Inc.
Written by John Gilmore, starting 1985-08-25.
@@ -227,6 +227,7 @@ static struct option long_options[] =
{"newer-mtime", required_argument, 0, NEWER_MTIME_OPTION},
{"null", no_argument, 0, NULL_OPTION},
{"no-anchored", no_argument, 0, NO_ANCHORED_OPTION},
+ {"no-deref-trans", no_argument, 0, 'a'},
{"no-ignore-case", no_argument, 0, NO_IGNORE_CASE_OPTION},
{"no-wildcards", no_argument, 0, NO_WILDCARDS_OPTION},
{"no-wildcards-match-slash", no_argument, 0,
NO_WILDCARDS_MATCH_SLASH_OPTION},
@@ -388,7 +389,9 @@ Archive format selection:\n\
stdout);
fputs (_("\
\n\
-Local file selection:\n\
+Local file selection:\n"), stdout);
+ fputs (_("\
+ -a --no-deref-trans dump the translator\n\
-C, --directory=DIR change to directory DIR\n\
-T, --files-from=NAME get names to extract or create from file NAME\n\
--null -T reads null-terminated names, disable -C\n\
@@ -456,7 +459,7 @@ or a file name starting with `/' or `.',
/* Parse the options for tar. */
-/* Available option letters are DEHIJQY and aenqy. Some are reserved:
+/* Available option letters are DEHIJQY and enqy. Some are reserved:
e exit immediately with a nonzero exit status if unexpected errors occur
E use extended headers (draft POSIX headers, that is)
@@ -467,7 +470,7 @@ or a file name starting with `/' or `.',
Y per-block gzip compression */
#define OPTION_STRING \
- "-01234567ABC:F:GIK:L:MN:OPRST:UV:WX:Zb:cdf:g:hijklmoprstuvwxyz"
+ "-01234567ABC:F:GIK:L:MN:OPRST:UV:WX:Zab:cdf:g:hijklmoprstuvwxyz"
static void
set_subcommand_option (enum subcommand subcommand)
@@ -592,6 +595,11 @@ decode_options (int argc, char **argv)
input_files++;
break;
+ case 'a':
+ /* Dump translators. */
+ dump_translators_option = 1;
+ break;
+
case 'A':
set_subcommand_option (CAT_SUBCOMMAND);
break;
diff -urp tar-1.13.25/src/tar.h tar-1.13.25_hurd/src/tar.h
--- tar-1.13.25/src/tar.h 2001-09-21 06:23:01.000000000 +0200
+++ tar-1.13.25_hurd/src/tar.h 2003-06-22 23:09:07.000000000 +0200
@@ -1,7 +1,7 @@
/* GNU tar Archive Format description.
Copyright (C) 1988, 1989, 1991, 1992, 1993, 1994, 1995, 1996,
- 1997, 2000, 2001 Free Software Foundation, Inc.
+ 1997, 2000, 2001, 2003 Free Software Foundation, Inc.
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
@@ -209,6 +209,12 @@ struct oldgnu_header
/* This file is a tape/volume header. Ignore it on extraction. */
#define GNUTYPE_VOLHDR 'V'
+/* Store Hurd translators. */
+#define GNUTYPE_TRANS 'T'
+
+/* Identifies the *next* file on the tape as having a long translatorname. */
+#define GNUTYPE_LONGTRANS 'A'
+
/* tar Header Block, overall structure. */
/* tar files are made in basic blocks of this size. */