[Bug 521730] Review Request: fossil - A distributed SCM with bug tracking and wiki

bugzilla at redhat.com bugzilla at redhat.com
Tue Nov 10 02:48:45 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=521730


Cristian Ciupitu <cristian.ciupitu at yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |cristian.ciupitu at yahoo.com




--- Comment #7 from Cristian Ciupitu <cristian.ciupitu at yahoo.com>  2009-11-09 21:48:43 EDT ---
(In reply to comment #5)
> Fossil uses copies of sqlite3.h and sqlite3.c that are bundled with the
> tarball. I fear it's necessary to remove them and use the development files
> from package sqlite-devel instead
> (http://fedoraproject.org/wiki/No_Bundled_Libraries). 
> 
> It's not too complicated to modify the sources. However, the code also
> references the internal sqlite function sqlite3StrICmp that is not available
> through libsqlite3. So it must probably be patched in somewhere.  

Indeed it's not too complicated to modify the sources. I have attached a patch
for the latest version of fossil (from the development repository), but it
works just as well with fossil-src-20091103194336.tar.gz (it's listed on the
"Download" page). The only catch is that it doesn't work with the current
sqlite-devel, but it does work sqlite-devel-3.6.17-1.fc11.x86_64 which is
available in the updates-testing repository. I assume that after some time it
will be available in the updates repository. Also, all the fossil automated
tests pass with 0 errors.


@Silas Sewell
If you still want to package fossil, don't forget to run "tclsh
src/makemake.tcl > src/main.mk" after applying the patch. Also, delete the
sqlite3 bundled files to be sure you're not compiling them.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the Fedora-package-review mailing list