Author Topic: The 30 August 2014 build (9884) is out.  (Read 61929 times)

Offline edison

  • Multiple posting newcomer
  • *
  • Posts: 53
Re: The 30 August 2014 build (9884) is out.
« Reply #45 on: September 16, 2014, 04:10:52 pm »
Thanks, it works.

Can you review and submit it ?

Code
Index: options_clang.xml
===================================================================
--- options_clang.xml (revision 9916)
+++ options_clang.xml (working copy)
@@ -185,7 +185,7 @@
         <Command name="LinkNative"
                  value="$linker $libdirs -o $exe_output $link_objects $link_resobjects $link_options $libs -Wl,--subsystem,native"/>
         <Command name="LinkExe"
-                 value="$linker $libdirs -o $exe_output $link_objects $link_resobjects $link_options $libs -mwindows"/>
+                 value="$linker $libdirs -o $exe_output $link_objects $link_resobjects $link_options $libs -Wl,--subsystem,windows"/>
         <Command name="LinkDynamic"
                  value="$linker -shared -Wl,--output-def=$def_output -Wl,--out-implib=$static_output -Wl,--dll $libdirs $link_objects $link_resobjects -o $exe_output $link_options $libs"/>
     </if>

Offline ollydbg

  • Developer
  • Lives here!
  • *****
  • Posts: 5910
  • OpenCV and Robotics
    • Chinese OpenCV forum moderator
If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13413
    • Travis build status
Re: The 30 August 2014 build (9884) is out.
« Reply #47 on: October 22, 2014, 10:27:42 pm »
@edison: Which version of clang is the last one that supported the old switch?
@alpha: Can we make this parse the version of clang and then pass the appropriate option?
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline stahta01

  • Lives here!
  • ****
  • Posts: 7582
    • My Best Post
Re: The 30 August 2014 build (9884) is out.
« Reply #48 on: October 23, 2014, 04:57:46 am »
@edison: Which version of clang is the last one that supported the old switch?
@alpha: Can we make this parse the version of clang and then pass the appropriate option?

I think going with the first clang version to support the new switch is also an option.

Tim S.
C Programmer working to learn more about C++ and Git.
On Windows 7 64 bit and Windows 10 64 bit.
--
When in doubt, read the CB WiKi FAQ. http://wiki.codeblocks.org

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: The 30 August 2014 build (9884) is out.
« Reply #49 on: October 23, 2014, 06:23:51 am »
Oops, must have missed this.  Yes, I should be able to add a quick regex.  Will do some testing this weekend to see what is necessary.

Offline edison

  • Multiple posting newcomer
  • *
  • Posts: 53
Re: The 30 August 2014 build (9884) is out.
« Reply #50 on: October 23, 2014, 06:39:20 am »
@edison: Which version of clang is the last one that supported the old switch?

I don't know...

http://comments.gmane.org/gmane.comp.compilers.clang.devel/38114

Reid Kleckner said:
Quote
I doubt we ever recognized this argument.  This probably changed because we started rejecting unknown -m flags.

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 13413
    • Travis build status
Re: The 30 August 2014 build (9884) is out.
« Reply #51 on: October 23, 2014, 07:16:51 am »
Oops, must have missed this.  Yes, I should be able to add a quick regex.  Will do some testing this weekend to see what is necessary.
Just keep in mind that dumpversion doesn't do anything useful - always returns 4.2.1 (at least on linux and osx)
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: The 30 August 2014 build (9884) is out.
« Reply #52 on: October 30, 2014, 02:32:45 am »
From my experimentation, it seems neither of these flags are necessary.
Code
         <Command name="LinkExe"
-                 value="$linker $libdirs -o $exe_output $link_objects $link_resobjects $link_options $libs -mwindows"/>
+                 value="$linker $libdirs -o $exe_output $link_objects $link_resobjects $link_options $libs -Wl,--subsystem,windows"/>

I will soon commit the equivalent of (for all versions of clang)
Code
        <Command name="LinkExe"
                 value="$linker $libdirs -o $exe_output $link_objects $link_resobjects $link_options $libs"/>
if there are no reported issues with it.

Offline edison

  • Multiple posting newcomer
  • *
  • Posts: 53
Re: The 30 August 2014 build (9884) is out.
« Reply #53 on: October 30, 2014, 09:40:33 am »
I will soon commit the equivalent of (for all versions of clang)
Code
        <Command name="LinkExe"
                 value="$linker $libdirs -o $exe_output $link_objects $link_resobjects $link_options $libs"/>
if there are no reported issues with it.

This will cause a console window below the gui window, make it not a "GUI application".

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: The 30 August 2014 build (9884) is out.
« Reply #54 on: October 30, 2014, 05:57:56 pm »
This will cause a console window below the gui window, make it not a "GUI application".
Oh dear... flawed test cases.  I am so embarrassed. :-[
I think I must be spending too much time under linux.

(Thank you.)