public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Warren Young <warren@etr-usa.com>
To: cygwin@cygwin.com
Subject: Re: Promote sqlite 3.7.13-1 from test status?
Date: Thu, 16 Aug 2012 12:48:00 -0000	[thread overview]
Message-ID: <502CE120.4050900@etr-usa.com> (raw)
In-Reply-To: <20120816105507.GD17546@calimero.vinschen.de>

On 8/16/2012 4:55 AM, Corinna Vinschen wrote:
> On Aug 16 04:30, Warren Young wrote:
>>
>> So, what you did with that requested change, Achim, is prevented
>> Cygwin svn from winning any fights over ownership of .svn/wc.db.
>
> So what?  Don't use native Windows tools in parallel accessing the
> same file.  Full stop.

I've been maintaining the Cygwin SQLite package for four years now. 
This recent wish for SQLite on Cygwin to act more Unix-like is the first 
such request I've received, and I don't remember it being an issue with 
the previous maintainer, either.

But, shortly after making the change -- because I didn't immediately see 
why it would hurt anything -- complaints started pouring in from actual 
people who've been doing what you say they shouldn't do.

So what's a package maintainer to do?  Go all WJM on them? :)

I don't see why I should break previously working behavior just out of 
purity, given that, so far, only one person ever in the entire world has 
been bothered enough by that lack of purity to ask me to change how 
SQLite is compiled.

> Why should mandatory locking be better here, while it's quite naturally
> using advisory locking in the entire UNIX world?

Because that's what a native Windows build of SQLite does.

(Why?  Because https://sqlite.org/faq.html#q5)

Advisory locking only works when all players cooperate.  We can't assume 
that on Windows, unless we set up an insular Cygwin ghetto.

> If you think it's absolutely necessary to use some Windows
> functions, what is it, and why?

The scenario is this:

Dev Fred likes to use the GUI TortoiseSVN client most of the time. 
(Fred is a little strange, but we like him anyway.)

There is no Cygwin port of TortoiseSVN, because it is a deeply 
Windows-native program, doing things like adding a shell extension to 
Windows Explorer.  There is also no Cygwin GUI SVN client we can 
recommend people use instead, and even if there were, you can imagine a 
lot of people would push back, given the consequent requirement for X11.

On occasion, Fred runs into a situation where the GUI can't do a given 
task, or at least can't do it as well as you can from the command line. 
  What are Fred's options?

Option 1: Download the native Windows Subversion port.  Sensible, but it 
means you have to use a crippled shell.

Option 2: Install Cygwin and add the svn package, so you have a 
reasonable command line environment.

Why should Option 2 result in incorrect behavior, when the correct fix 
is known and -- much more important -- already implemented upstream?

Maybe you didn't realize that last, Corinna: Cygwin SQLite is built 
pretty much stock OOTB.  There's just one minor patch I make, which 
replaces a deprecated Cygwin 1.5 DLL call with its v1.7 replacement. 
Aside from ifdefs and such, it's about as stock a build as you could 
wish.  It's not like I'm patching a bunch of Windowsisms into the 
upstream source purely for Cygwin's sake.

Upstream's already made the call.

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

  reply	other threads:[~2012-08-16 12:02 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-13 17:20 [ANNOUNCEMENT] Updated: sqlite3-3.7.13-1 Warren Young
2012-08-16  3:38 ` Promote sqlite 3.7.13-1 from test status? (was: Updated: sqlite3-3.7.13-1) Warren Young
2012-08-16  4:56   ` Promote sqlite 3.7.13-1 from test status? Larry Hall (Cygwin)
2012-08-16  9:01   ` Promote sqlite 3.7.13-1 from test status? (was: Updated: sqlite3-3.7.13-1) Achim Gratz
2012-08-16  9:03     ` Corinna Vinschen
2012-08-16 11:06       ` Promote sqlite 3.7.13-1 from test status? Warren Young
2012-08-16 11:39         ` Corinna Vinschen
2012-08-16 12:48           ` Warren Young [this message]
2012-08-16 13:32             ` Corinna Vinschen
2012-08-16 14:32               ` Earnie Boyd
2012-08-16 16:20               ` Warren Young
2012-08-16 17:46                 ` Brian Wilson
2012-08-16 18:15                   ` David Rothenberger
2012-08-16 18:37                   ` Warren Young
2012-08-16 22:29                     ` James Johnston
2012-08-17  3:52                       ` Yaakov (Cygwin/X)
2012-08-17  3:56                         ` David Rothenberger
2012-08-17 20:03                         ` Brian Wilson
2013-02-05 19:51                         ` Achim Gratz
2013-02-11 20:07                           ` Warren Young
2013-02-11 21:24                             ` Achim Gratz
2013-02-11 21:51                               ` Warren Young
2013-02-11 22:00                                 ` Achim Gratz
2013-02-12  6:26                                   ` Warren Young
2013-02-12 17:54                                     ` Achim Gratz
2013-03-01 15:12                                     ` Achim Gratz
2013-03-01 22:52                                       ` Warren Young
2013-03-02  7:53                                         ` Achim Gratz
2013-02-11 22:49                                 ` David Stacey
2012-08-17 19:49                       ` Brian Wilson
2012-08-17  1:40                     ` JonY
2012-08-16 20:41               ` Options for getting mandatory locking in cygwin1.dll (was: Promote sqlite 3.7.13-1 from test status?) Warren Young
2012-08-17  9:45                 ` Corinna Vinschen
2012-08-17 14:25                   ` Andrey Repin
2012-08-17 15:52                   ` Options for getting mandatory locking in cygwin1.dll Warren Young
2012-08-16 15:06             ` Promote sqlite 3.7.13-1 from test status? Thrall, Bryan
2012-08-17  9:09             ` Andrey Repin
2012-11-17  5:15     ` Daniel Colascione
2012-08-16  9:04   ` Promote sqlite 3.7.13-1 from test status? (was: Updated: sqlite3-3.7.13-1) Adam Dinwoodie
2012-11-20 12:51 ` [ANNOUNCEMENT] Updated: sqlite3-3.7.13-1 Yaakov (Cygwin/X)
2012-11-21 18:43   ` Warren Young
2012-11-21 19:02     ` Achim Gratz
2012-11-23 22:13       ` Jon Lambert
2013-01-08 21:31       ` sqlite3-3.7.15.1-1 packages to test Warren Young
2013-01-08 21:36         ` Achim Gratz
2013-01-09 22:28         ` David Stacey
2013-01-10  0:08           ` Warren Young
2013-01-20  9:23         ` David Stacey
2013-01-20 10:45           ` Achim Gratz
2013-01-20 13:01             ` David Stacey
2013-02-06 22:44               ` David Stacey
2013-02-11 20:14                 ` Warren Young
2013-02-11 22:23                   ` David Stacey
2012-11-21 19:04     ` [ANNOUNCEMENT] Updated: sqlite3-3.7.13-1 Christopher Faylor
2012-11-21 19:40     ` David Stacey
2012-11-22 16:43       ` Christopher Faylor

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=502CE120.4050900@etr-usa.com \
    --to=warren@etr-usa.com \
    --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).