Re: [Geany-Devel] [geany/geany] f03206: Windows: Compile geany_private.rc into Geany.exe for version data and icon
On 19/04/15 15:40, Enrico Tröger wrote: > On 18/04/15 21:35, Matthew Brush wrote: >> On 15-04-18 03:51 AM, Enrico Tröger wrote: >>> On 18/04/15 11:47, Enrico Tröger wrote: On 17/04/15 21:32, Colomban Wendling wrote: > Le 12/04/2015 22:54, Enrico Tröger a écrit : >> Branch: refs/heads/master >> Author: Enrico Tröger >> Committer: Enrico Tröger >> Date:Sun, 12 Apr 2015 20:54:16 UTC >> Commit: f03206744361baac1c811db97f5d2816409d7787 >> >> https://github.com/geany/geany/commit/f03206744361baac1c811db97f5d2816409d7787 >> >> >> Log Message: >> --- >> Windows: Compile geany_private.rc into Geany.exe for version data >> and icon > > Should we integrate geany_private.rc in the library too? This isn't > done with Autotools, but could be added. It's not really necessary. For Waf, I just added it to the binary where it is necessary and kept it for the library. The rc file serves two main purposes: - add the icon to the binary - add versioning information to the binary (e.g. in the file properties dialog in the Details tab) While especially the icon is important for the binary, I guess the rc file is just nice to have for the library but not necessary. But I agree we should do the same for Waf and autotools, either link it or don't link it. I don't mind which way to go. >>> >>> After thinking a bit more about this, I think we should not link the rc >>> file against the library. We also don't link it against the plugin >>> libraries. >>> >>> The rc file contains settings that the resulting binary is an executable >>> and it includes also the geany.exe.manifest. Both do not apply to the >>> library. So it is actually wrong to use that rc file for the library. >>> >>> If there are no objections, I will remove linking the rc file against >>> the library in Waf. >>> >> >> None here. >> >> I think the way it might make sense is if it contained artwork, UIs, >> translations and stuff like they often do for Win32 API projects, and >> that were actually used in libgeany.dll, but since we just use it for >> the .exe icon, meta-info, styles, etc. I tend to agree it makes more >> sense linked to the .exe. >> >> We might still want an .rc file for the library though, if it's >> important to show the correct meta-info in Windows explorer for the .dll >> file (I dunno if it is?). > > I don't think so. The only benefit I see is that in the Windows explorer > file properties dialog version information would appear. I don't know if > Windows or any related software rely on this information or use it in > any way. > > We also don't add this information to the plugin libraries and so why do > it for geany.dll. To finish this, I just removed linking the rc file against the library. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc signature.asc Description: OpenPGP digital signature ___ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Re: [Geany-Devel] [geany/geany] f03206: Windows: Compile geany_private.rc into Geany.exe for version data and icon
On 18/04/15 21:35, Matthew Brush wrote: > On 15-04-18 03:51 AM, Enrico Tröger wrote: >> On 18/04/15 11:47, Enrico Tröger wrote: >>> On 17/04/15 21:32, Colomban Wendling wrote: Le 12/04/2015 22:54, Enrico Tröger a écrit : > Branch: refs/heads/master > Author: Enrico Tröger > Committer: Enrico Tröger > Date:Sun, 12 Apr 2015 20:54:16 UTC > Commit: f03206744361baac1c811db97f5d2816409d7787 > > https://github.com/geany/geany/commit/f03206744361baac1c811db97f5d2816409d7787 > > > Log Message: > --- > Windows: Compile geany_private.rc into Geany.exe for version data > and icon Should we integrate geany_private.rc in the library too? This isn't done with Autotools, but could be added. >>> >>> It's not really necessary. For Waf, I just added it to the binary where >>> it is necessary and kept it for the library. >>> The rc file serves two main purposes: >>> - add the icon to the binary >>> - add versioning information to the binary (e.g. in the file properties >>> dialog in the Details tab) >>> >>> While especially the icon is important for the binary, I guess the rc >>> file is just nice to have for the library but not necessary. >>> But I agree we should do the same for Waf and autotools, either link it >>> or don't link it. >>> I don't mind which way to go. >> >> After thinking a bit more about this, I think we should not link the rc >> file against the library. We also don't link it against the plugin >> libraries. >> >> The rc file contains settings that the resulting binary is an executable >> and it includes also the geany.exe.manifest. Both do not apply to the >> library. So it is actually wrong to use that rc file for the library. >> >> If there are no objections, I will remove linking the rc file against >> the library in Waf. >> > > None here. > > I think the way it might make sense is if it contained artwork, UIs, > translations and stuff like they often do for Win32 API projects, and > that were actually used in libgeany.dll, but since we just use it for > the .exe icon, meta-info, styles, etc. I tend to agree it makes more > sense linked to the .exe. > > We might still want an .rc file for the library though, if it's > important to show the correct meta-info in Windows explorer for the .dll > file (I dunno if it is?). I don't think so. The only benefit I see is that in the Windows explorer file properties dialog version information would appear. I don't know if Windows or any related software rely on this information or use it in any way. We also don't add this information to the plugin libraries and so why do it for geany.dll. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc signature.asc Description: OpenPGP digital signature ___ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Re: [Geany-Devel] [geany/geany] f03206: Windows: Compile geany_private.rc into Geany.exe for version data and icon
On 15-04-18 03:51 AM, Enrico Tröger wrote: On 18/04/15 11:47, Enrico Tröger wrote: On 17/04/15 21:32, Colomban Wendling wrote: Le 12/04/2015 22:54, Enrico Tröger a écrit : Branch: refs/heads/master Author: Enrico Tröger Committer: Enrico Tröger Date:Sun, 12 Apr 2015 20:54:16 UTC Commit: f03206744361baac1c811db97f5d2816409d7787 https://github.com/geany/geany/commit/f03206744361baac1c811db97f5d2816409d7787 Log Message: --- Windows: Compile geany_private.rc into Geany.exe for version data and icon Should we integrate geany_private.rc in the library too? This isn't done with Autotools, but could be added. It's not really necessary. For Waf, I just added it to the binary where it is necessary and kept it for the library. The rc file serves two main purposes: - add the icon to the binary - add versioning information to the binary (e.g. in the file properties dialog in the Details tab) While especially the icon is important for the binary, I guess the rc file is just nice to have for the library but not necessary. But I agree we should do the same for Waf and autotools, either link it or don't link it. I don't mind which way to go. After thinking a bit more about this, I think we should not link the rc file against the library. We also don't link it against the plugin libraries. The rc file contains settings that the resulting binary is an executable and it includes also the geany.exe.manifest. Both do not apply to the library. So it is actually wrong to use that rc file for the library. If there are no objections, I will remove linking the rc file against the library in Waf. None here. I think the way it might make sense is if it contained artwork, UIs, translations and stuff like they often do for Win32 API projects, and that were actually used in libgeany.dll, but since we just use it for the .exe icon, meta-info, styles, etc. I tend to agree it makes more sense linked to the .exe. We might still want an .rc file for the library though, if it's important to show the correct meta-info in Windows explorer for the .dll file (I dunno if it is?). Cheers, Matthew Brush ___ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Re: [Geany-Devel] [geany/geany] f03206: Windows: Compile geany_private.rc into Geany.exe for version data and icon
On 18/04/15 11:47, Enrico Tröger wrote: > On 17/04/15 21:32, Colomban Wendling wrote: >> Le 12/04/2015 22:54, Enrico Tröger a écrit : >>> Branch: refs/heads/master >>> Author: Enrico Tröger >>> Committer: Enrico Tröger >>> Date:Sun, 12 Apr 2015 20:54:16 UTC >>> Commit: f03206744361baac1c811db97f5d2816409d7787 >>> >>> https://github.com/geany/geany/commit/f03206744361baac1c811db97f5d2816409d7787 >>> >>> Log Message: >>> --- >>> Windows: Compile geany_private.rc into Geany.exe for version data and icon >> >> Should we integrate geany_private.rc in the library too? This isn't >> done with Autotools, but could be added. > > It's not really necessary. For Waf, I just added it to the binary where > it is necessary and kept it for the library. > The rc file serves two main purposes: > - add the icon to the binary > - add versioning information to the binary (e.g. in the file properties > dialog in the Details tab) > > While especially the icon is important for the binary, I guess the rc > file is just nice to have for the library but not necessary. > But I agree we should do the same for Waf and autotools, either link it > or don't link it. > I don't mind which way to go. After thinking a bit more about this, I think we should not link the rc file against the library. We also don't link it against the plugin libraries. The rc file contains settings that the resulting binary is an executable and it includes also the geany.exe.manifest. Both do not apply to the library. So it is actually wrong to use that rc file for the library. If there are no objections, I will remove linking the rc file against the library in Waf. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc signature.asc Description: OpenPGP digital signature ___ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Re: [Geany-Devel] [geany/geany] f03206: Windows: Compile geany_private.rc into Geany.exe for version data and icon
On 17/04/15 21:32, Colomban Wendling wrote: > Le 12/04/2015 22:54, Enrico Tröger a écrit : >> Branch: refs/heads/master >> Author: Enrico Tröger >> Committer: Enrico Tröger >> Date:Sun, 12 Apr 2015 20:54:16 UTC >> Commit: f03206744361baac1c811db97f5d2816409d7787 >> >> https://github.com/geany/geany/commit/f03206744361baac1c811db97f5d2816409d7787 >> >> Log Message: >> --- >> Windows: Compile geany_private.rc into Geany.exe for version data and icon > > Should we integrate geany_private.rc in the library too? This isn't > done with Autotools, but could be added. It's not really necessary. For Waf, I just added it to the binary where it is necessary and kept it for the library. The rc file serves two main purposes: - add the icon to the binary - add versioning information to the binary (e.g. in the file properties dialog in the Details tab) While especially the icon is important for the binary, I guess the rc file is just nice to have for the library but not necessary. But I agree we should do the same for Waf and autotools, either link it or don't link it. I don't mind which way to go. > Though, `waf build -v` warns that geany_private.rc.o is created more > than once (doesn't seem to be a real problem though). Noticed this too but yeah, I think too this isn't a big problem. Ideally Waf should detect that it built the file already for another target and just use the built object. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc signature.asc Description: OpenPGP digital signature ___ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel
Re: [Geany-Devel] [geany/geany] f03206: Windows: Compile geany_private.rc into Geany.exe for version data and icon
Le 12/04/2015 22:54, Enrico Tröger a écrit : > Branch: refs/heads/master > Author: Enrico Tröger > Committer: Enrico Tröger > Date:Sun, 12 Apr 2015 20:54:16 UTC > Commit: f03206744361baac1c811db97f5d2816409d7787 > > https://github.com/geany/geany/commit/f03206744361baac1c811db97f5d2816409d7787 > > Log Message: > --- > Windows: Compile geany_private.rc into Geany.exe for version data and icon Should we integrate geany_private.rc in the library too? This isn't done with Autotools, but could be added. Though, `waf build -v` warns that geany_private.rc.o is created more than once (doesn't seem to be a real problem though). ___ Devel mailing list Devel@lists.geany.org https://lists.geany.org/cgi-bin/mailman/listinfo/devel