public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Achim Gratz <Stromeko@nexgo.de>
To: cygwin@cygwin.com
Subject: Re: [ANNOUNCEMENT] Updated: sqlite3-3.7.13-1
Date: Wed, 21 Nov 2012 19:02:00 -0000	[thread overview]
Message-ID: <878v9udble.fsf@Rainer.invalid> (raw)
In-Reply-To: <50AD20AD.2090908@etr-usa.com>

Warren Young writes:
> On 11/20/2012 05:51, Yaakov (Cygwin/X) wrote:
>> On Mon, 2012-08-13 at 10:12 -0600, Warren Young wrote:
>>> This is a *test* version which reverts the patch added to 3.7.12.1-1,
>>> which caused problems with Subversion as a side effect, particularly in
>>> systems using non-Cygwin programs with their svn checkout trees.
>>
>> And in so doing, removed several APIs which were added in 3.7.12.1,
>> breaking other packages which used them once they were made available.
>> Compatibility with Windows shouldn't come before backwards compatibility
>> for Cygwin packages.
>
> Okay, that makes a second vote for principle in four months.  (The
> other being Corinna's.)  Add to that the original Achim Gratz vote for
> "Unix mode," which carries more weight since it was accompanied by an
> actual example of problems.

FWIW, I think Yaakov is more immediately concerned about the additional
API:

 -DSQLITE_ENABLE_COLUMN_METADATA\
 -DSQLITE_ENABLE_FTS3 -DSQLITE_ENABLE_FTS_PARENTHESIS\
 -DSQLITE_ENABLE_FTS4

some of which you've been removing together with the reversion to the
use of the Windows API, maybe even inadvertently.

> As far as I'm concerned, Cygwin SQLite will remain built in Cygwin
> mode until Cygwin has a way to conditionally request mandatory locks,
> so that we may have our cake and eat it, too.

Right.  I've not been able to spend much time on it and the windows code
is structured differently enough so it's hard to compare, but if the
database gets locked from a windows application the POSIX API runs into
some sort of timeout that SQLite interprets as a non-recoverable disk
error.  Either there should be another (POSIX) API that recognizes the
lock or when the particular error happens it should be checked if the
file is locked — maybe with a windows API, but certainly conditionally
just for Cygwin.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Wavetables for the Terratec KOMPLEXER:
http://Synth.Stromeko.net/Downloads.html#KomplexerWaves


--
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-11-21 19:02 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-13 17:20 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
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 [this message]
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=878v9udble.fsf@Rainer.invalid \
    --to=stromeko@nexgo.de \
    --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).