Re: [ITP] mingw-w64 Second try

2010-09-11 Thread JonY

On 9/10/2010 08:51, JonY wrote:

On 9/10/2010 07:09, Charles Wilson wrote:

On 9/9/2010 6:10 AM, JonY wrote:

OK, we're amost there.

binutils and runtime are GTG.

gcc, headers, and pthreads are really close.

Everything rebuilds from source fine, and the uploaded packages actually
match the rebuilt versions (or vice versa). So that's all good. Plus, I
was able to build a sample project using these tools
(mingw64-x86_86-xz-4.999.9beta-1) with no problems.

However, when I tried to install these packages via setup.exe, I ran
into problems with the setup.hints. In the first case, genini (*) did
not like some of them -- complained about missing fields -- AND, genini
couldn't actually parse the package name of mingw64-x86_64-headers.

(*) genini is a user script that can be used to generate a setup.ini
for a cygwin package repository. On the sourceware server, a different
tool -- upset -- is used.

Now, maybe -- MAYBE -- upset is smarter than genini, and won't complain.
However, it's usually better to make sure that genini is happy; then you
are much more likely to NOT cause the upset script on sourceware to send
cgf a bunch of nasty warnings via email. That makes cgf angry. You
wouldn't like cgf when he's angry.

Here are the bad setup.hints:

mingw64-x86_64-gcc-core/setup.hint
mingw64-x86_64-gcc-fortran/setup.hint
mingw64-x86_64-gcc-g++/setup.hint
mingw64-x86_64-gcc-objc/setup.hint
mingw64-x86_64-gcc-ada/setup.hint
mingw64-x86_64-gcc/setup.hint

mingw64-x86_64-pthreads/setup.hint

Plus there are bigger changes required to the -headers package. We'll
talk about that last.

In each of the following cases, the setup.hint was missing an ldesc:
field. This annoys genini, but upset might be ok with it.
mingw64-x86_64-gcc-core/setup.hint
mingw64-x86_64-gcc-fortran/setup.hint
mingw64-x86_64-gcc-g++/setup.hint
mingw64-x86_64-gcc-objc/setup.hint
mingw64-x86_64-gcc-ada/setup.hint
mingw64-x86_64-pthreads/setup.hint

The top gcc setup.hint was missing both an ldesc: and a requires:
field (requires was empty, of course).
mingw64-x86_64-gcc/setup.hint

I've attached the updated setup.hints.

Here's what I would suggest we do, for gcc: DON'T bother to rebuild it.
Just save these setup.hints, and make sure to fold them in to your NEXT
official release's CYGWIN-PATCHES/ directory. When I upload this first
version of mingw64-gcc, I'll be sure to use these new hints instead of
the ones you pasted inline. Let's call gcc GTG with alternate hint
files.

For pthreads, I'd suggest you actually rebuild and re-upload it with the
attached hint (it doesn't take nearly as long...) but if you want to
treat it just like gcc, above, that's fine. Let me know -- we'll call
pthreads GTG with alternate hint files, or as rebuilt.

Now...headers.

Here's the problem: genini requires that the version part of the
package name begin with a numeral. I think this is true of upset as
well, but I'm not sure (but again, better safe than sorry. You wouldn't
like cgf when he's angry). Also, genini and cygport do NOT like having
a hyphen in the version number (but upset is ok with it).

So, just liike your earlier gendef and libmangle packages -- where we
discovered that they had to be named, e.g.

gendef-1.0-svn2931-1.tar.bz2
gendef-1.0-svn2931-1-src.tar.bz2

libmangle-1.0-svn2930-1.tar.bz2
libmangle-1.0-svn2930-1-src.tar.bz2

...the current name of the headers package is no good:

mingw64-x86_64-headers-svn3433-1.tar.bz2
mingw64-x86_64-headers-svn3433-1-src.tar.bz2
^^^
must start with numeral

Now, following the gendef/libmangle pattern, I tried

mingw64-x86_64-headers-1.0b-svn3433-1.tar.bz2

where 1.0b came from the configure.ac file, but I discovered that genini
doesn't like the hyphen between 1.0b and svn3433 -- when you do that,
genini (and cygport-0.10.0, for that matter) thinks the package name is
mingw64-x86_64-headers-1.0b, and the version is svn3433, and we're
right back where we started! Now, upset doesn't care about this --
obviously -- since gendef and libmangle, which use '-' between 1.0 and
svn -- have not yet caused cgf to become angry.

But...let's also try to keep genini happy. And, it may be a regression
in cygport -- I'm not sure (obv. you were able to build gendef and
libmangle with an older version of cygport, even with a hyphen in the
middle of the version string), but we have to keep cygport happy, too.

So, I rebuilt as:

mingw64-x86_64-headers-1.0b_svn3433-1.tar.bz2
mingw64-x86_64-headers-1.0b_svn3433-1-src.tar.bz2




OK, this is fine.


And that worked fine (genini was happy, cygport was happy, setup.exe was
happy -- and presumably upset will be happy == no angry cgf). I've
uploaded the modified version here:

http://cygwin.cwilson.fastmail.fm/mingw64-x86_64-headers-1.0b_svn3433-1-src.tar.bz2

http://cygwin.cwilson.fastmail.fm/mingw64-x86_64-headers-1.0b_svn3433-1.tar.bz2


so you can download it and pick it apart. The only thing that's
different is (a) the package name, (b) the name of the 

Re: setup, upx, and TLS

2010-09-11 Thread Lapo Luchini
[previous message was sent to the wrong mailing list; a nice reminder
not to write emails at 5:27 in the night...]

Yaakov (Cygwin/X) wrote:
 On Sun, 2010-09-05 at 15:27 -0400, Charles Wilson wrote:
 Lapo, are you still here?  Could we get an updated upx package, please?
 
 I'm not so sure that he is still here.

Sorry guys, a long strike of too many things to do at the same time had
me going a bit in rounds.

I've packaged upx (I hope not to have forgot anything of the process
;)), but I fear I'll have the next decently sized time-slot on monday
(uh, that'll be a problem to send the announce mesage earlier too).

Oh, and I had ready-to-release packages of monotone and tidy, though the
first I'm fairly sure it's good, while the latter was copied directly
from CygPorts but I didn't really have time to look into the way the
package is divided in three and I'd prefer uploading it in a few days
(same goes for other packages).

http://lapo.it/cygwin/upx/setup.hint
http://lapo.it/cygwin/upx/upx-3.07-1-src.tar.bz2
http://lapo.it/cygwin/upx/upx-3.07-1.tar.bz2

http://lapo.it/cygwin/monotone/monotone-0.48-1-src.tar.bz2
http://lapo.it/cygwin/monotone/monotone-0.48-1.tar.bz2
http://lapo.it/cygwin/monotone/setup.hint

-- 
Lapo Luchini - http://lapo.it/

“C is quirky, flawed, and an enormous success.” (Dennis M. Ritchie)

“The Magic Words are Squeamish Ossifrage” (Ron Rivest, RSA-129
challenge, 1977)


Re: setup, upx, and TLS

2010-09-11 Thread Corinna Vinschen
On Sep 11 10:23, Lapo Luchini wrote:
 http://lapo.it/cygwin/upx/setup.hint
 http://lapo.it/cygwin/upx/upx-3.07-1-src.tar.bz2
 http://lapo.it/cygwin/upx/upx-3.07-1.tar.bz2
 
 http://lapo.it/cygwin/monotone/monotone-0.48-1-src.tar.bz2
 http://lapo.it/cygwin/monotone/monotone-0.48-1.tar.bz2
 http://lapo.it/cygwin/monotone/setup.hint

Both packages uploaded.


Thanks,
Corinna

-- 
Corinna Vinschen  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader  cygwin AT cygwin DOT com
Red Hat


setup.exe compilation fix

2010-09-11 Thread Václav Haisman
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256

Hi.

I just tried to compile setup.exe from source and run into a problem in
propsheet.cc. The names like PropSheet_SetCurSel() are macros and already
contain the `::' resulting into `' and compile error. The attached patch
fixes it for me. Changelog below.

- -- 
VH


2010-09-10  Václav Haisman  v.hais...@sh.cvut.cz

* propsheet.cc (PropSheet::SetActivePage): Remove :: before 
PropSheet_SetCurSel.
(PropSheet::SetActivePageByID): Remove :: before 
PropSheet_SetCurSelByID.
(PropSheet::SetButtons): Remove :: before PropSheet_SetWizButtons.
(PropSheet::PressButton): Remove :: before PropSheet_PressButton.

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iF4EAREIAAYFAkyLkMcACgkQeqrf2dJjGj41mQD8C6nc9hM8rBY9fdsvI6bV0OLb
aCliFrdPSSLJ66gIWUMA/0m4Fh3yUglWAv6vxmDwrHo28FWn+A7wbiLgzTOUm6yQ
=/PMJ
-END PGP SIGNATURE-
Index: propsheet.cc
===
RCS file: /cvs/cygwin-apps/setup/propsheet.cc,v
retrieving revision 2.15
diff -u -p -r2.15 propsheet.cc
--- propsheet.cc30 Jun 2009 04:14:29 -  2.15
+++ propsheet.cc11 Sep 2010 14:14:33 -
@@ -441,7 +441,7 @@ bool
 PropSheet::SetActivePage (int i)
 {
   // Posts a message to the message queue, so this won't block
-  return static_cast  bool  (::PropSheet_SetCurSel (GetHWND (), NULL, i));
+  return static_cast  bool  (PropSheet_SetCurSel (GetHWND (), NULL, i));
 }
 
 bool
@@ -449,18 +449,18 @@ PropSheet::SetActivePageByID (int resour
 {
   // Posts a message to the message queue, so this won't block
   return static_cast  bool 
-(::PropSheet_SetCurSelByID (GetHWND (), resource_id));
+(PropSheet_SetCurSelByID (GetHWND (), resource_id));
 }
 
 void
 PropSheet::SetButtons (DWORD flags)
 {
   // Posts a message to the message queue, so this won't block
-  ::PropSheet_SetWizButtons (GetHWND (), flags);
+  PropSheet_SetWizButtons (GetHWND (), flags);
 }
 
 void
 PropSheet::PressButton (int button)
 {
-  ::PropSheet_PressButton (GetHWND (), button);
+  PropSheet_PressButton (GetHWND (), button);
 }


propsheet.cc.diff.sig
Description: Binary data