This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v2] Added file properties to windows gdb executable for all mingw32 builds.
- From: Eli Zaretskii <eliz at gnu dot org>
- To: "Bunk, Bernd" <bernd dot bunk at intel dot com>
- Cc: tromey at redhat dot com, palves at redhat dot com, gdb-patches at sourceware dot org
- Date: Fri, 23 Aug 2013 17:37:27 +0300
- Subject: Re: [PATCH v2] Added file properties to windows gdb executable for all mingw32 builds.
- References: <1377161766-8318-1-git-send-email-bernd dot bunk at intel dot com> <8361uxkbi9 dot fsf at gnu dot org> <E2C54CDFBA86B845B3B075E2B2042A91202566CA at IRSMSX106 dot ger dot corp dot intel dot com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> From: "Bunk, Bernd" <bernd.bunk@intel.com>
> CC: "tromey@redhat.com" <tromey@redhat.com>, "palves@redhat.com"
> <palves@redhat.com>, "gdb-patches@sourceware.org"
> <gdb-patches@sourceware.org>
> Date: Fri, 23 Aug 2013 13:44:58 +0000
>
> > > +# check for environment variables to replace certain file properties
> > > +[ -n "$WIN_EXE_VERSION" ] && version=$WIN_EXE_VERSION [ -n
> > > +"$WIN_EXE_COMPANY_NAME" ] && company_name=$WIN_EXE_COMPANY_NAME
> > > +[ -n "$WIN_EXE_FILE_DESCRIPTION" ] &&
> > > +file_description=$WIN_EXE_FILE_DESCRIPTION
> > > +[ -n "$WIN_EXE_PRODUCT_NAME" ] && product_name=$WIN_EXE_PRODUCT_NAME
> > > +[ -n "$WIN_EXE_INTERNAL_NAME" ] &&
> > > +internal_name=$WIN_EXE_INTERNAL_NAME
> > > +[ -n "$WIN_EXE_ORIGINAL_FILENAME" ] &&
> > > +original_filename=$WIN_EXE_ORIGINAL_FILENAME
> > > +[ -n "$WIN_EXE_COPYRIGHT" ] && copyright=$WIN_EXE_COPYRIGHT [ -n
> > > +"$WIN_EXE_LICENSE" ] && license=$WIN_EXE_LICENSE [ -n
> > > +"$WIN_EXE_CONFIGURED" ] && configured=$WIN_EXE_CONFIGURED [ -n
> > > +"$WIN_EXE_SUPPORT" ] && support=$WIN_EXE_SUPPORT
> >
> > This looks like unnecessary featurism to me. Is it really needed, and
> > if so, in what use cases?
> Yes, it is needed. Not in here, but for every company which changes/adds and re-distributes gdb.
> I started this feature because our Product Validation does not like binaries without legal information.
> And off course this is different depending on who ships the product.
> Without a way to change the strings the complete changeset would be useless for me.
You can always modify the source of these attributes, can't you? It's
not like you change these strings several times a day, right?
> > > +#include "afxres.h"
> >
> > Is this header really needed?
> Yes, unfortunately.
> This header file "implements" the resource language used below in the RC file.
Sorry, I don't understand: this header file in MinGW distribution just
includes windows.h, defines IDC_STATIC, and that's it. What do you
have in it?