Build patch
Monitor your SVN repositories and notifies you on new commits
Brought to you by:
steveking
Menu
▾
▴
1 Attachments
#361 Build patch
Originally created by: XhmikosR
Originally owned by: tortoisesvn
Hi. Attached is a patch which includes the following changes:
1) Removes invalid info from build.txt and default.build regarding VS2008.
2) Uses consistently /O2 and /GL /LTCG. If you don't want to use /LTCG you should remove both /GL and /LTCG from all *.build files. At the moment some files have /LTCG specified but not /GL; this fixes a couple of compiler/linker warnings
3) Removes invalid switches for lib.exe (/DEBUG /PDB) which fixes a couple of linker warnings
4) Specifies /Oy since VS2010 doesn't enable it even with /O2 specified although the documentation says it is enabled.
5) Adds /OPT:ICF where it's missing
6) Removes empty properties from CommitMonitor.vcxproj and fixes linking with the debug sqlite.lib even for release builds
7) Adds the -f switch in versioninfo.build for SubWCRev since some times when you update the external repositories for instance the revision number won't be updated
I have this patch for so long and I thought I'd post it. Feel free to adapt any changes you need/want.
Thanks for all your hard work!
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: tortoisesvn
(No comment was entered for this change.)
Labels: Milestone-1.8.2
Owner: tortoisesvn
Status: Started
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: tortoisesvn
(No comment was entered for this change.)
Labels: -Type-Defect Type-Enhancement
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: tortoisesvn
thanks!
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: tortoisesvn
This issue was closed by revision [r650].
Status: Fixed
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: XhmikosR
Thanks for committing the patch. It seems you missed the /Oy change in CommitMonitor.vcxproj. So while I was at it I removed the default option ResourceOutputFileName and set SetChecksum-true for Release builds since sometimes the checksum isn't added when using /LTCG.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: tortoisesvn
This issue was updated by revision [r652].
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: XhmikosR
This should be the last patch. :)
1) updated the info in build.txt and replaced any VS2008 references to VS2010 also in default.build.user.tmpl
2) tabs->to spaces for consistency among the *.build files
3) /MACHINE:X64 for consistency even though it's not really used atm
4) added /RELEASE for the sasl dlls
5) move machineoption above <lib> in subversion.build since it's closer to where the option is used
6) compiles OpenSSL with /O2 /Oy /GL and /LTCG. If you don't want /LTCG for OpenSSL just remove /GL and /LTCG, the rest changes should be kept.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: tortoisesvn
This issue was updated by revision [r653].