New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clarify purpose of fetching #64
Comments
From jamilan on August 18, 2009 00:48:43 Could be this approach better than adding the defines to sqlite.c? In setup.py
|
From jamilan on August 18, 2009 01:12:14 Actually this will also work if not fetching. In setup.py
+if not sys.platform in ('win32', 'win64'):
Look for amalgamation in our directory or in sqlite3 subdirectoryamalgamation=( |
From rogerbinns on August 24, 2009 04:46:50 I don't really want to go down this path as it is creeping featuritis, but could be I do intend to add a new setup command (ie the proper way of doing things) so that |
From jamilan on September 23, 2009 02:58:03 We just update to the lastest code and here is an update patch, a cleaner one I must {{{ This includes the functionality marked as experimental in SQLite 3.Comment out the line to exclude themdefine_macros.append( ('EXPERIMENTAL', '1') )
End of customizations@@ -204,12 +207,10 @@
}}} We are using the same procedure than the EXPERIMENTAL flag, and it does not modify Can we do something else to convince you? |
From rogerbinns on September 23, 2009 14:18:57 Doing what you want is relatively trivial from a code point of view. (I'd take a But what isn't clear to me is why you insist on using the APSW fetch functionality if I am mostly convinced to not modify the .c file for one reason - it makes it harder |
From jamilan on September 24, 2009 03:19:12 Our workflow for apsw/sqlite integrations is the following: We update a priscine copy of apsw from mercurial and run the next command on it: The current configuration of apsw and sqlite works perfectly for us, and We do not need to track every change that happened on sqlite or apsw since On our repository (we are using git) we need a stable and operative apsw´s version, When they update their sources, all they have to do is: We know the proposed change may not fit everybody, but we found it easy to implement But as always it is only a proposal, and we do not want to force anyone to use it. ;) |
From rogerbinns on September 24, 2009 03:29:55 You didn't answer my question about why you insist on using APSW's fetch There is a problem of where to put the various HAVE_* definitions since Makefiles Note that you could supply the HAVE_* defs to the setup.py build_ext command either |
From jamilan on September 24, 2009 04:44:04 Well, the fetch functionality does the same as a manual download and it is automatic, We do not care about the version of sqlite in use, since we always use it through We even compile our sqlite extensions over the sqlite fetch done by apsw and The sqlite3config.h solution sound good, I will test it as soon as you place it on |
From rogerbinns on September 25, 2009 02:52:26 It is now committed to Mercurial as revision 12dc7e1efe The documentation (look in doc/build.rst) still says that setup.py modifies the files Let me know if things work for you now so I can close this. Status: Fixed |
From jamilan on September 28, 2009 02:15:21 Works for us, thanks!! |
From rogerbinns on September 28, 2009 02:17:24 Verified fixed by jamilan Status: Verified |
From rogerbinns on August 14, 2009 09:43:01
See http://www.sqlite.org/cvstrac/tktview?tn=4026
Original issue: http://code.google.com/p/apsw/issues/detail?id=64
The text was updated successfully, but these errors were encountered: