Hi Erik, thanks for the information. Now I also understand your constraints 😊
I think I'll then try to come up with some configure flag for setting JDK_RC_NAME. And I'll see if I can do it in a way that you would not need to change something in your internal setup. /Christoph > -----Original Message----- > From: Erik Joelsson <erik.joels...@oracle.com> > Sent: Mittwoch, 3. April 2019 16:18 > To: Langer, Christoph <christoph.lan...@sap.com>; build- > d...@openjdk.java.net > Subject: Re: RFR (S): 8221880: Better customization for Windows RC > properties FileDescription and ProductName > > Hello Christoph, > > I understand your problem, but a complicating factor here is that the > version-numbers file is currently formatted as a properties file and we > do consume it as such in other places. While we don't specifically look > for this property there, I think it sets a bad precedent if we let it > become a shell script instead. Could you find a solution without > variable references in version-numbers? > > We do override PRODUCT_NAME for our builds, but we do not do it by > patching the version-numbers file. We do it through the custom extension > hooks in configure. With your change here, that would no longer work > unless we override this new JDK_RC_NAME variable explicitly. > > I guess I would be OK with JDK_RC_NAME="OpenJDK Platform", but we will > need a corresponding internal fix very quickly, so please keep me > updated when such a change is pushed. > > /Erik > > On 2019-04-03 01:06, Langer, Christoph wrote: > > Hi, > > > > In our downstream build, I'd like to be able to set/customize the value for > the Windows RC properties "ProductName" and "FileDescription" via the > version-numbers file. These values manifest in Windows executable > properties. > > > > During the build ProductName gets set to "OpenJDK Platform 13" and > FileDescription will be "OpenJDK Platform binary". This value is obtained by > concatenating \$(PRODUCT_NAME) \$(JDK_RC_PLATFORM_NAME) in flags- > other.m4. Both variables get set in version-numbers. So, if I was to customize > the properties, I could change PRODUCT_NAME and > JDK_RC_PLATFORM_NAME in version-numbers. However, modifying the > former is no good idea since it is used ubiquitously and has unwanted side > effects. On the other hand, I could make an adaption to flags-other.m4, but > that diff would be hidden and not in a central place where I'd expect such > customizing diffs. > > > > So, please review this small fix, which allows for modifying these RC > properties in version-numbers. The default behavior won't be changed. > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8221880 > > Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8221880.0/ > > > > Thanks > > Christoph > >
