[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Update to SQLite amalgamation instructions.

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Fri, 28 Oct 2011 11:35:50 -0400

Daniel Shahaf <d.s_at_daniel.shahaf.name> writes:
>Daniel Shahaf wrote on Fri, Oct 28, 2011 at 09:19:37 +0200:
>> Karl Fogel wrote on Thu, Oct 27, 2011 at 23:28:43 -0400:
>> > It's been a while, so out of caution I'd appreciate review on this
>> > before I commit, even though it's a fairly trivial patch. Thanks!
>> >
>> Why did you drop the direct link to the .tgz? The log message doesn't
>> say, and I know that elsewhere

Because the URL was broken (see log msg). And it seems likely that no
version-specific URL for this is likely to stay valid for long.

However, if that was the only reason we were computing the URL, then the
computation should be removed too. I'll take a look.



>> we do some printf(1) magic to compute it.
>> > -SQLITE_URL="http://www.sqlite.org/sqlite-amalgamation-${SQLITE_RECOMMENDED_VER}.tar.gz"
>> > +SQLITE_AMALGAMATION_URL="http://sqlite.org/download.html"
Received on 2011-10-28 17:36:26 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.