Re: [PATCH] Update to SQLite amalgamation instructions.

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
>
>get-deps.sh

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.

Thanks!

-Karl

>> 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