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: "Bunk, Bernd" <bernd dot bunk at intel dot com>
- To: Eli Zaretskii <eliz at gnu dot org>
- Cc: "tromey at redhat dot com" <tromey at redhat dot com>, "palves at redhat dot com" <palves at redhat dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Mon, 26 Aug 2013 12:24:50 +0000
- Subject: RE: [PATCH v2] Added file properties to windows gdb executable for all mingw32 builds.
- Authentication-results: sourceware.org; auth=none
- 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> <83eh9kiic8 dot fsf at gnu dot org>
> -----Original Message-----
> From: Eli Zaretskii [mailto:eliz@gnu.org]
> Sent: Friday, August 23, 2013 4:37 PM
> To: Bunk, Bernd
> Cc: tromey@redhat.com; palves@redhat.com; gdb-patches@sourceware.org
> Subject: Re: [PATCH v2] Added file properties to windows gdb executable
> for all mingw32 builds.
>
> > 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?
If I have to change/overwrite the sources for this feature just to use the feature, where would be the reason for me to upstream it? I need parameters/env vars to customize the behavior during build, not changed sources that I need to merge for every branch I have. The whole purpose of upstreaming a feature is to NOT have custom sources for this feature.
>
> > > > +#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?
As clarified in another feedback - my mistake. I changed it to 'include "version.h"' in v3. I also tested it with using Visual Studio on this RC file - still functional.
Intel GmbH
Dornacher Strasse 1
85622 Feldkirchen/Muenchen, Deutschland
Sitz der Gesellschaft: Feldkirchen bei Muenchen
Geschaeftsfuehrer: Christian Lamprechter, Hannes Schwaderer, Douglas Lusk
Registergericht: Muenchen HRB 47456
Ust.-IdNr./VAT Registration No.: DE129385895
Citibank Frankfurt a.M. (BLZ 502 109 00) 600119052