From: cgf AT cygnus DOT com (Christopher Faylor)
Subject: Re: Another winsup snapshot 981116
16 Nov 1998 20:19:22 -0800
Message-ID: <19981116231103.A16090.cygnus.cygwin32.developers@cygnus.com>
References: <19981116190521 DOT 36874 AT cygnus DOT com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
To: cygwin32-developers AT cygnus DOT com

Just a plea to anyone who reads this.

Please don't send in reports that configures are really slow unless you
can provide some insight.

With the last several snapshots and with B20 itself we've received a lot
of reports from people who excitedly noticed things that were mentioned
in Geoff's announcement mail.

Just in case it isn't clear, here are some things I, personally,
do not want to hear:

1) Configures do not seem to run very quickly.

2) Configures are really so slow that I can't use them.

3) Configures were slow so I dropped back to B18.

4) Configures are very much like a tortoise.

5) I wrote a shell script and put a lot of commands in it.  I use
   many pipes and backticks in it to store values in variables. Then
   I use sed to massage stuff like Makefiles.
   It's really slow.

6) I know you mentioned something about configures being slow but
   I wanted to let you know that they are slow here too.  I'm using
   Windows on a PC.  Hope this helps!

7) I think I figured out why configure is so slow:  It's taking a long
   time for the commands to execute.

8) I tried putting a .bat extension on my configure script.  It wasn't slow
   but it didn't seem to work quite right.

9) My cofnigure seems to be hung.  I've let it run for two horus and it
   isnt done yet.

10) How do I strip the \r's from a file?  I don't want them.  I think this
    should be handled automatically so I don't have to worry about it.
    I can't see why they don't do this.

Hope this helps.

-chris

On Mon, Nov 16, 1998 at 07:05:21PM -0800, Geoffrey Noer wrote:
>Here's another winsup snapshot.  This one is getting close to ready
>for a 20.1 release.  Configures work on both NT and Windows 98 for me
>but they're *really* slow (4 hours instead of 10-20 minutes).  But on
>some machines this seems not to be a problem which is making it hard
>for us to find the culprit.  Obviously this needs to be fixed before
>we release the update.  ChangeLog entries since the last one follow
>as usual.
>
>-- 
>Geoffrey Noer
>noer AT cygnus DOT com
>
>----------------------------> snip <-------------------------------
>
>Mon Nov 16 16:15:20 1998  Geoffrey Noer  <noer AT cygnus DOT com>
>
>        * Makefile.in: FLAGS_TO_PASS should pass CFLAGS and CXXFLAGS
>        not ALL_CFLAGS and ALL_CXXFLAGS.  Rework handling of CFLAGS
>        variables to remove duplication and make it possible to
>        build mingw directory.
>
>Mon Nov 16 09:40:21 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * syscalls.cc (_open): Use new macro to check for
>        NULL or empty pathname and return appropriate error.
>        * path.cc (check_null_empty_path*): Move macro to
>        path.h.
>        * path.h: Move macros here.
>
>Sun Nov 15 20:23:10 1998  Geoffrey Noer  <noer AT cygnus DOT com>
>
>        * include/cygwin/version.h: up minor API version to mark
>        addition of dll_noncygwin_dllcrt0 to exports list.
>
>Sun Nov 15 23:05:21 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * path.cc (check_null_empty_path): New macro.
>        (check_null_empty_path_errno): Ditto.
>        (path_conv::path_conv): Check for NULL and empty
>        path names here and set appropriate error.
>        (conv_to_win32_path): Ditto.
>        (conv_to_full_win32_path): Ditto.
>        (conv_to_posix_path): Ditto.
>        (conv_to_full_posix_path): Ditto.
>        (mount_info::conv_wo_win32_path): Back out previous
>        change.
>        (mount_info::conv_to_posix_path): Ditto.
>
>Sun Nov 15 19:29:19 1998  Geoffrey Noer  <noer AT cygnus DOT com>
>
>        * include/limits.h: define CHILD_MAX to be 63.
>        * sysconf.cc (sysconf): return CHILD_MAX when asked for
>        instead of _POSIX_CHILD_MAX.
>
>Sun Nov 15 18:25:22 1998  Geoffrey Noer  <noer AT cygnus DOT com>
>
>        patch from Mumit Khan  <khan AT xraylith DOT wisc DOT edu>:
>        * path.cc (mount_info::conv_to_posix_path): Handle NULL and
>        empty pathnames.
>        (mount_info::conv_to_win32_path): Likewise.
>
>Sun Nov 15 18:09:06 1998  Geoffrey Noer  <noer AT cygnus DOT com>
>
>        * cygwin.din: export dll_noncygwin_dllcrt0.
>        * {libccrt0.cc, winsup.h}: minor reformat.
>
>Sat Nov 14 22:14:51 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * environ.cc (environ_init): Use sizeof for string lengths
>        rather than hard-coded values.
>
>        patch from Mumit Khan <khan AT xraylith DOT wisc DOT edu>:
>        * dll_init.cc (dll_foreign_dllcrt0): Rename to
>        dll_noncygwin_dllcrt0.
>        * winsup.h: Ditto.
>        * include/cygwin/cygwin_dll.h: Ditto.
>
>Sat Nov 14 14:09:33 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * hinfo.cc (hinfo::de_linearize_fd_array): Reset first fd
>        for open to zero after an exec.
>
>Sat Nov 14 01:29:23 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * fork.cc (fork): No need for intermediate jmp_buf variable.
>
>        patch from Mumit Khan <khan AT xraylith DOT wisc DOT edu>:
>        * libccrt0.cc (cygwin_attach_foreign_dll): New function.
>        * dll_init.cc (dll_foreign_dllcrt0): New function to initialize
>        DLLs loaded by non-cygwin apps.
>        (dll_dllcrt0_1): Initialize process table entry.
>        (dll_dllcrt0): Revert last change to do partial initialization.
>        * winsup.h (dll_foreign_dllcrt0): Prototype.
>        * include/cygwin/cygwin_dll.h (_cygwin_foreign_dll_entry): New
>        entry point that does partial initialization for non-cygwin apps.
>
>Fri Nov 13 16:17:28 1998  Geoffrey Noer  <noer AT cygnus DOT com>
>
>        Assorted -Wall cleanup:
>        * utils/mount.cc (show_mounts): remove unused var i.
>        * utils/cygcheck.cc (dll_info): comment out unused var ofs.
>        (dump_sysinfo): return a void, not an int.  Comment out unused
>        var len in two places.  Add parens around assignment used as
>        truth value.
>        * utils/kill.cc (main): remove unused var i
>        * utils/mkpasswd: include <ctype.h> and <stdlib.h>.
>        (enum_users): remove unused var rc.
>        (enum_local_groups): ditto.
>        * utils/ps.cc (main): Add parens around assignments used as
>        truth values.
>        * utils/mkgroup.c: include <ctype.h> and <stdlib.h>.
>        (enum_groups): return void, not int.  Remove unused var rc.
>        * gmon.c: include <stdio.h> and <unistd.h>.
>        (_mcleanup): remove unused vars buf and profdir.
>
>Fri Nov 13 16:17:28 1998  Geoffrey Noer  <noer AT cygnus DOT com>
>
>        patch from Corinna Vinschen <corinna DOT vinschen AT cityweb DOT de>:
>        * include/sys/sysmacros.h: new file, define major, minor, makedev
>
>Fri Nov 13 17:03:52 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * dcrt0.cc: Reflect a variable name change.
>        * fhandler.h: *::ready_for read takes an additional argument.
>        * fhandler_console.cc (get_non_ascii_key): New function derived
>        from fhandler_console::read1.
>        (fhandler_console::read1): Use new function.
>        * select.cc (*::ready_for_read): Add new "nonblocking" argument.
>        (peek_console): Be more diligent in detecting when a usable
>        character has been entered or false positives will be returned.
>        (fhandler_tty_common::ready_for_read): Detect attempt to perform
>        a "background" read.
>        * syscalls.cc (read_handler): Remove code for dealing with slow
>        devices.  Just deal with non-blocking here.
>        (read_ready_thread): Renamed function.  Rewritten to only detect
>        when an fd has data ready to be read.  Actual reads happen in
>        the main thread.
>        (_read): Rely on read_handler to read data for everything.  Use
>        the read_ready_thread to indicate when data is ready to be read.
>        Gut the read_thread_info class since it no longer needs to return
>        much information.
>        * winsup.h: Reflect above changes to read_thread_info.  Rename
>        to read_ready_thread_info.
>
>Fri Nov 13 15:09:26 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        patch from Corinna Vinschen <corinna DOT vinschen AT cityweb DOT de> and
>        Ron Parker <rdparker AT butlermfg DOT org>:
>        * path.cc (mount_info::conv_to_win32_path): Refined recognition of
>        UNC devices.
>
>Fri Nov 13 12:37:00 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * Makefile.in: Fix a typo.
>
>Fri Nov 13 10:59:43 1998  DJ Delorie  <dj AT cygnus DOT com>
>
>        * Makefile.in: "make" uses new "ld -shared" by default internally.
>
>Fri Nov 13 00:58:38 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * spawn.cc (find_exec): Make sure that return value is correct
>        in all cases.  Document.
>        * fhandler.h (fhandler_tty_common): Give unit number to constructor.
>        * fhandler_tty.cc (fhandler_tty_slave::fhandler_tty_slave): Give
>        unit number to descriptor.  Create correct UNIX path name for
>        tty so that ttyname() will work correctly.
>        * select.cc (fhandler_tty_common::ready_for_read): Don't worry
>        about read_for_read since the tty read routines are already
>        interruptible.
>        * syscalls.cc (_read): Reset read thread events on signal.
>
>        patch from Mumit Khan <khan AT xraylith DOT wisc DOT edu>:
>        * dlfcn.cc (ctype.h): Include.
>        (check_access): Document.
>        (check_path_access): Document.
>        (get_full_path_to_dll): Rework to handle general filenames and
>        symlinks.
>
>Thu Nov 12 17:01:52 1998  DJ Delorie  <dj AT cygnus DOT com>
>
>        * Makefile.in (shared): "make shared" uses the new "ld -shared"
>        * cygwin.din: add LIBRARY and BASE tags for "ld -shared"
>
>Wed Nov 11 17:16:17 1998  Christopher Faylor <cgf AT cygnus DOT com>
>
>        * configure.in: Add --enable options for various cygwin defines
>        including preliminary thread-safe defines.
>        * configure: Regenerate
>        * Makefile.in: Honor new --enable options.
>        * glob.h: Move BSD defines to sys/cdefs.h.
>        * include/sys/cdefs.h: Move __P definition here.

-- 
cgf AT cygnus DOT com
http://www.cygnus.com/