Euphoria Ticket #661: minGW euc source/buildsys.e should link to comctrl32 not comdlg32

~InitCommonControlsEX requires link to comctrl32 for minGW gcc. this also works for watcom although watcom seems to find comctrl32 ok if you link to comdlg32

# HG changeset patch 
# User ne1uno <ne1uno@users.sourceforge.net> 
# Date 1304352000 14400 
# Branch 4.0 
# Node ID b74df49afe47881d889aae3dcd3e67d280ab5e12 
# Parent  f606b0ae83dd0cdf222e715d46c2b38e98824db1 
fix buildsys to use comctrl32 instead of comdlg32 like the makefiles 
 
diff -r f606b0ae83dd -r b74df49afe47 source/buildsys.e 
--- a/source/buildsys.e	Mon May 02 11:56:01 2011 -0400 
+++ b/source/buildsys.e	Mon May 02 12:00:00 2011 -0400 
@@ -432,7 +432,7 @@ 
 			elsif TOSX then 
 				l_flags &= " -lresolv" 
 			elsif TWINDOWS then 
-				l_flags &= " -mno-cygwin -lws2_32 -lcomdlg32"				 
+				l_flags &= " -mno-cygwin -lws2_32 -lcomctl32"				 
 			end if 
 			 
 			-- input/output 
@@ -465,7 +465,7 @@ 
 				end if 
 			end if 
  
-			l_flags &= sprintf(" FILE %s LIBRARY ws2_32 LIBRARY comdlg32", { user_library }) 
+			l_flags &= sprintf(" FILE %s LIBRARY ws2_32 LIBRARY comctl32", { user_library }) 
 			 
 			 
 			-- resource file, executable file  

Details

Type: Bug Report Severity: Normal Category: Translator
Assigned To: mattlewis Status: Fixed Reported Release: 4.0.2
Fixed in SVN #: View VCS: none Milestone: 4.0.3

1. Comment by mattlewis May 05, 2011

See: hg:euphoria/rev/5c63702c9ffc

changeset: 4886:5c63702c9ffc branch: debugger user: Matt Lewis date: Thu May 05 14:45:25 2011 -0400 files: source/buildsys.e description:

  • fix buildsys to link correct windows dll for translated programs
  • fixes ticket 661

2. Comment by mattlewis May 05, 2011

See: hg:euphoria/rev/b2d1b87e91c0

changeset: 4887:b2d1b87e91c0 branch: debugger user: Matt Lewis date: Thu May 05 14:45:35 2011 -0400 files: docs/release/4.0.3.txt description:

  • release notes for ticket 661

3. Comment by mattlewis May 05, 2011

See: hg:euphoria/rev/1f1d9358be0d

changeset: 4888:1f1d9358be0d branch: 4.0 parent: 4882:f557348317e8 user: Matt Lewis date: Thu May 05 14:46:53 2011 -0400 files: docs/release/4.0.3.txt source/buildsys.e description:

  • fix ticket 661 in 4.0 branch

4. Comment by ne1uno May 11, 2011

tried to reply to commit msg but it's read only

Date: Wed, 11 May 2011 13:02:52 -0400 
Subject: Re: [Rapideuphoria-commits] euphoria: * removed some 4.1 code that was unintentionally added 
buildsys.e 
                       c_flags &= sprintf(" -c -w -fsigned-char -O2 -m%d -I%s -ffast-math", 
-                               { sizeof( C_POINTER ) * 8, get_eucompiledir() }) 
+                               { get_eucompiledir() }) 
 
should'nt this be: 
+                               { 32, get_eucompiledir() }) 
 

5. Comment by mattlewis May 12, 2011

Oops. My editor had been upgraded, and some settings got reset (namely, checking for external modifications), and didn't get all of the resulting junk cleaned up.

See hg:euphoria/rev/9af47aa6ef56 for the update.

6. Comment by SDPringle Jul 02, 2011

This is fixed.

7. Comment by mattlewis Jul 13, 2011

See: hg:euphoria/rev/751e95981026

changeset: 5062:751e95981026 branch: 4.0 parent: 5060:1f136c2188ca user: Matt Lewis date: Wed Jul 13 05:44:11 2011 -0400 files: docs/release/4.0.4.txt description:

  • ticket 661 was fixed in (and included in release notes for) 4.0.3

Search



Quick Links

User menu

Not signed in.

Misc Menu