A resource file often references in other files, but there currently is no
automated way of building these depency.
This patch add 'deps' as an optionally output format to windres which can be
used to do exactly this.
Index: binutils/resrc.c
===================================================================
RCS file: /cvs/src/src/binutils/resrc.c,v
retrieving revision 1.35
diff -u -b -B -u -r1.35 resrc.c
--- binutils/resrc.c 30 Jul 2008 04:34:56 -0000 1.35
+++ binutils/resrc.c 22 May 2009 17:06:23 -0000
@@ -723,6 +723,7 @@
e = open_file_search (filename, FOPEN_RB, "bitmap file", &real_filename);
+ if (!e) return;
if (stat (real_filename, &s) < 0)
fatal (_("stat failed on bitmap file `%s': %s"), real_filename,
strerror (errno));
@@ -766,6 +767,7 @@
e = open_file_search (filename, FOPEN_RB, "cursor file", &real_filename);
+ if (!e) return;
/* A cursor file is basically an icon file. The start of the file
is a three word structure. The first word is ignored. The
second word is the type of data. The third word is the number of
@@ -962,6 +964,7 @@
e = open_file_search (filename, FOPEN_RB, "font file", &real_filename);
+ if (!e) return;
if (stat (real_filename, &s) < 0)
fatal (_("stat failed on font file `%s': %s"), real_filename,
strerror (errno));
@@ -1168,6 +1171,7 @@
e = open_file_search (filename, FOPEN_RB, "icon file", &real_filename);
+ if (!e) return;
/* The start of an icon file is a three word structure. The first
word is ignored. The second word is the type of data. The third
word is the number of entries. */
@@ -1498,6 +1502,8 @@
e = open_file_search (filename, FOPEN_RB, "messagetable file",
&real_filename);
+ if (!e) return;
+
if (stat (real_filename, &s) < 0)
fatal (_("stat failed on bitmap file `%s': %s"), real_filename,
strerror (errno));
@@ -1720,6 +1726,7 @@
e = open_file_search (filename, FOPEN_RB, "file", &real_filename);
+ if (!e) return;
if (stat (real_filename, &s) < 0)
fatal (_("stat failed on file `%s': %s"), real_filename,
strerror (errno));
Index: binutils/windres.c
===================================================================
RCS file: /cvs/src/src/binutils/windres.c,v
retrieving revision 1.38
diff -u -b -B -u -r1.38 windres.c
--- binutils/windres.c 9 Feb 2009 15:40:24 -0000 1.38
+++ binutils/windres.c 22 May 2009 17:06:24 -0000
@@ -71,7 +71,9 @@
/* Binary RES file. */
RES_FORMAT_RES,
/* COFF file. */
- RES_FORMAT_COFF
+ RES_FORMAT_COFF,
+ /* Dependencies list for make rules - we can only sanely use this for
output*/
+ RES_FORMAT_DEPSONLY
};
/* A structure used to map between format types and strings. */
@@ -89,6 +91,7 @@
{ "rc", RES_FORMAT_RC },
{ "res", RES_FORMAT_RES },
{ "coff", RES_FORMAT_COFF },
+ { "dep", RES_FORMAT_DEPSONLY },
{ NULL, RES_FORMAT_UNKNOWN }
};
@@ -101,6 +104,7 @@
{ "exe", RES_FORMAT_COFF },
{ "obj", RES_FORMAT_COFF },
{ "o", RES_FORMAT_COFF },
+ { "d", RES_FORMAT_DEPSONLY },
{ NULL, RES_FORMAT_UNKNOWN }
};
@@ -132,6 +136,9 @@
static void target_put_16 (void *, rc_uint_type);
static rc_uint_type target_get_32 (const void *, rc_uint_type);
static void target_put_32 (void *, rc_uint_type);
+static void write_deps_file (const char * filename);
+static void read_deps_file (void);
+static void add_fname (const char*);
/* When we are building a resource tree, we allocate everything onto
@@ -181,8 +188,38 @@
return obstack_alloc (&reswr_obstack, (size_t) bytes);
}
+
+/* This structure is used for storing referenced filenames */
+struct dependant
+{
+ /* next name */
+ struct dependant *next;
+ /* file name */
+ char *name;
+};
+
+/* Keep the head of the dependecies list to hand */
+static struct dependant *dep_list = NULL;
+
+/* Extend dependencies list */
+void
+add_fname(s)
+ const char* s;
+{
+struct dependant *item;
+
+ item = xmalloc(sizeof (struct dependant));
+ item->name = xstrdup(s);
+ item->next = dep_list;
+ dep_list=item;
+
+}
+
+
/* Open a file using the include directory search list. */
+int missingfiles=0;
+
FILE *
open_file_search (const char *filename, const char *mode, const char *errmsg,
char **real_filename)
@@ -194,6 +231,7 @@
if (e != NULL)
{
*real_filename = xstrdup (filename);
+ add_fname(filename);
return e;
}
@@ -209,6 +247,8 @@
if (e != NULL)
{
*real_filename = n;
+
+ add_fname(n);
return e;
}
@@ -217,9 +257,10 @@
}
}
- fatal (_("can't open %s `%s': %s"), errmsg, filename, strerror (errno));
+ fprintf (stderr, _("can't open %s `%s': %s\n"), errmsg, filename, strerror
(errno));
- /* Return a value to avoid a compiler warning. */
+ add_fname(filename);
+ missingfiles =1;
return NULL;
}
@@ -688,7 +729,7 @@
-h --help Print this help message\n\
-V --version Print version information\n"));
fprintf (stream, _("\
-FORMAT is one of rc, res, or coff, and is deduced from the file name\n\
+FORMAT is one of rc, res, dep or coff, and is deduced from the file name\n\
extension if not specified. A single file name is an input file.\n\
No input-file is stdin, default rc. No output-file is stdout, default
rc.\n"));
@@ -1031,11 +1072,17 @@
case RES_FORMAT_COFF:
resources = read_coff_rsrc (input_filename, target);
break;
+ case RES_FORMAT_DEPSONLY:
+ read_deps_file ();
+ break;
}
if (resources == NULL)
fatal (_("no resources"));
+ if (missingfiles && (output_format != RES_FORMAT_DEPSONLY))
+ fatal("resources incomplete");
+
/* Sort the resources. This is required for COFF, convenient for
rc, and unimportant for res. */
resources = sort_resources (resources);
@@ -1056,12 +1103,48 @@
case RES_FORMAT_COFF:
write_coff_file (output_filename, target, resources);
break;
+ case RES_FORMAT_DEPSONLY:
+ write_deps_file (output_filename);
+ break;
}
xexit (0);
return 0;
}
+void
+read_deps_file ()
+{
+ fatal ("It is not possible to read deps files usefully");
+}
+
+
+void
+write_deps_file (const char *filename)
+{
+ FILE *e;
+ struct dependant *list;
+
+ if (filename == NULL)
+ e = stdout;
+ else
+ {
+ e = fopen (filename, FOPEN_WT);
+ if (e == NULL)
+ fatal ("can't open `%s' for output: %s", filename, strerror (errno));
+ }
+
+ list=dep_list;
+ while(list) {
+ fprintf(e," %s",list->name);
+ list=list->next;
+ }
+ fprintf(e,"\n");
+ fclose (e);
+}
+
+
+
static int
find_arch_match(const char *tname,const char **arch)
{
--
Summary: windres should be able to generate deps.
Product: binutils
Version: unspecified
Status: NEW
Severity: enhancement
Priority: P2
Component: binutils
AssignedTo: unassigned at sources dot redhat dot com
ReportedBy: rgammans at computer-surgery dot co dot uk
CC: bug-binutils at gnu dot org
GCC target triplet: i586-mingw32
http://sourceware.org/bugzilla/show_bug.cgi?id=10188
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils
