From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-104586-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 25125 invoked by alias); 23 Aug 2013 14:37:26 -0000
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
Received: (qmail 25108 invoked by uid 89); 23 Aug 2013 14:37:26 -0000
X-Spam-SWARE-Status: No, score=-4.1 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_DNSWL_NONE,RCVD_IN_HOSTKARMA_NO,SPF_SOFTFAIL autolearn=no version=3.3.2
Received: from mtaout20.012.net.il (HELO mtaout20.012.net.il) (80.179.55.166)    by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Fri, 23 Aug 2013 14:37:25 +0000
Received: from conversion-daemon.a-mtaout20.012.net.il by a-mtaout20.012.net.il (HyperSendmail v2007.08) id <0MRZ00G00N77PC00@a-mtaout20.012.net.il> for gdb-patches@sourceware.org; Fri, 23 Aug 2013 17:37:23 +0300 (IDT)
Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout20.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0MRZ00GAANAANC40@a-mtaout20.012.net.il>; Fri, 23 Aug 2013 17:37:22 +0300 (IDT)
Date: Fri, 23 Aug 2013 14:37:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [PATCH v2] Added file properties to windows gdb executable for all mingw32 builds.
In-reply-to: <E2C54CDFBA86B845B3B075E2B2042A91202566CA@IRSMSX106.ger.corp.intel.com>
To: "Bunk, Bernd" <bernd.bunk@intel.com>
Cc: tromey@redhat.com, palves@redhat.com, gdb-patches@sourceware.org
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <83eh9kiic8.fsf@gnu.org>
References: <1377161766-8318-1-git-send-email-bernd.bunk@intel.com> <8361uxkbi9.fsf@gnu.org> <E2C54CDFBA86B845B3B075E2B2042A91202566CA@IRSMSX106.ger.corp.intel.com>
X-SW-Source: 2013-08/txt/msg00680.txt.bz2

> 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?