public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Charles Wilson <cygwin@cwilson.fastmail.fm>
To: cygwin@cygwin.com
Subject: Re: [1.7] rebaseall doesn't solve the problem
Date: Wed, 04 Mar 2009 14:59:00 -0000	[thread overview]
Message-ID: <49AE9742.4070108@cwilson.fastmail.fm> (raw)
In-Reply-To: <20090304111849.GB10046@calimero.vinschen.de>

Corinna Vinschen wrote:
> Success!
> 
> - I removed all Cygwin traces from my TS test machine.
> - I disabled DEP for all applications and rebooted (necessary so that the
>   next step succeeds).
> - I installed a 1.7 distro from scratch.
> - I re-enabled DEP for all apps and rebooted.
> - I disabled the TS hack in the Cygwin DLL, rebuilt it and installed it
>   on the TS machine.
> - I started bash and it crashed.
> - I started ash (always works for some reason).
> - I called "./peflags -t 1 /bin/bash"

But it seems that the rebase package itself, when built, must be sure to
mark peflags.exe as tsaware before packaging it for distribution,
regardless of which ld.exe is used.  Just in case -- otherwise the poor
TS user won't be able to run peflags to fix everything else?

That leads to a small chicken-and-egg problem if somebody tries to build
the rebase package on a TS system, before the new ld is available.  But
we can live with that.

> - I started bash and... it worked!
> - I started GDB and it crashed.
> - I called "./peflags -t 1 /bin/gdb"
> - I started GDB and it worked.
> - I started grep and...
> - I called ... /bin/grep
> - ... worked.
> - ... and ssh ...
> - ... and mkpasswd ...
> - ... and mkgroup ...

Glad to hear it.

> So it seems that setting the TS-aware flag for all applications by
> default is the way to go, same as in Visual C++.

Makes sense.

> Unfortunately that doesn't help us with existing packages in the distro,
> only for new ones, but at least we now have the peflags tool for these
> cases.

Well, we will soon.  There's still a few things to deal with.
  1) code audit. I'd appreciate a thorough review of the code before
recommending that people use this tool on every app in their
installation.  I know, no warranty express or implied, We're Just Mean,
and You Get To Keep The Pieces, but still...
  2) peflagsall
  3) I'd like to make sure that the interface to peflags and the new
option(s) to ld are the same or very similar.  Once Dave posts his patch
for ld to the binutils list, I'm sure there will be much bike-shedding
about the name of the options, and perhaps discussion about the
implementation. I'd like to wait for ld CVS to be updated with Dave's
eventual contribution after those issues are resolved, and then make
whatever changes to peflags seem appropriate before releasing the tool
"officially".
  4) testing, testing, testing.  Naturally.

Until then, keep the URL to my previous message in this thread handy:
"Download peflags.c from ... and ..." <g>

> Nevertheless, I will disable the TS hack in Cygwin 1.7 now.  We have a
> much better solution now and the hack was a real PITA performance-wise.

I guess, as long as that URL "solution/FAQ answer" is sufficient in the
interim.

> Thank you very much for doing that!  I think this deserves a gold star.
> Igor?  Are you still with us?  I didn't see a mail from you since October.

I'm glad you're pleased with the tsaware facilities, but frankly I'm
happier that the dynbase feature SEEMS to reduce the incidence of fork
problems under Vista!  My previously-working 1.7 installation became
downright unusable last week before I wrote this tool.

--
Chuck

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

  reply	other threads:[~2009-03-04 14:59 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-21  2:27 Charles Wilson
2009-02-24 10:06 ` Corinna Vinschen
2009-02-27 21:22   ` Charles Wilson
2009-02-28 10:43     ` Corinna Vinschen
2009-02-28 18:50       ` Charles Wilson
2009-02-28 19:34         ` Matt Wozniski
2009-02-28 19:51         ` Christopher Faylor
2009-02-28 20:04           ` Charles Wilson
2009-02-28 20:29           ` Corinna Vinschen
2009-02-28 21:30             ` Charles Wilson
2009-03-01  9:47               ` Corinna Vinschen
2009-03-01 10:05                 ` Corinna Vinschen
2009-03-02 12:08             ` Corinna Vinschen
2009-03-02 15:14               ` Charles Wilson
2009-03-03  1:45                 ` Dave Korn
2009-03-03  2:06                   ` Charles Wilson
2009-03-03  2:44                     ` Dave Korn
2009-03-03 14:31                       ` Dave Korn
2009-03-03  4:28                   ` Christopher Faylor
2009-03-03  4:51                     ` Dave Korn
2009-03-03  5:07                       ` Christopher Faylor
2009-03-03  5:46                         ` Dave Korn
2009-02-28 20:16         ` Corinna Vinschen
2009-02-28 21:19           ` Charles Wilson
2009-03-01 10:20             ` Corinna Vinschen
2009-03-02  3:52               ` Charles Wilson
2009-03-04  6:00               ` Charles Wilson
2009-03-04  6:01                 ` Charles Wilson
2009-03-04  8:49                   ` Corinna Vinschen
2009-03-04 11:19                     ` Corinna Vinschen
2009-03-04 14:59                       ` Charles Wilson [this message]
2009-03-04 15:30                         ` Corinna Vinschen
2009-03-05  3:39                           ` peflags utility [Was: Re: [1.7] rebaseall doesn't solve the problem] Charles Wilson
2009-03-05  3:55                             ` Dave Korn
2009-03-16  6:33                             ` peflags utility Charles Wilson
2009-03-16 12:43                               ` Corinna Vinschen
2009-03-17  6:49                               ` Charles Wilson
2009-03-10  0:30               ` [1.7] rebaseall doesn't solve the problem Matthew Woehlke
2009-03-10 13:34                 ` Charles Wilson
2009-03-10 18:39                   ` Matthew Woehlke
2009-03-11  3:55                   ` ABCD
2009-04-09  3:18 Jonathan
2009-04-09  3:55 Charles Wilson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49AE9742.4070108@cwilson.fastmail.fm \
    --to=cygwin@cwilson.fastmail.fm \
    --cc=cygwin@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).