DINKIN / heekscad

Automatically exported from code.google.com/p/heekscad
Other
0 stars 0 forks source link

allow use of system tinyxml library #239

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Linux distributions like Fedora or Debian require software package to use
system copies of libraries instead using copies included in the source
code. The attached patch adds an option to use system copy of tinyxml
library when built with USE_SYSTEM_TINYXML. When a plain build is used,
then nothing changes.

It will be also nice when the OBJECTS line in the Makefile would be split
into multiple lines, ideally one object one line. It would beeasier to
track the changes there.

Original issue reported on code.google.com by d...@danny.cz on 18 Jan 2010 at 3:18

Attachments:

GoogleCodeExporter commented 9 years ago
I like this patch.  I think we should apply it.

tinyxml is in Debian Sid (but not in Squeeze).  It is not in any Ubuntu distro 
up to
and including Lucid.  I don't know for sure, but I'd guess tinyxml is not 
available
on Mac or Win32, which are the other platforms people use HeeksCAD on.  So 
while this
patch is a step in the right direction, it's of limited practical value.

It would be useful to identify a good small cross-platform XML library and 
switch to it.

Original comment by seb.kuzm...@gmail.com on 20 Mar 2010 at 3:11

GoogleCodeExporter commented 9 years ago
tinyxml is working ok, but I don't mind changing to another XML library, if it 
works
ok too.

Original comment by danhe...@gmail.com on 20 Mar 2010 at 9:42

GoogleCodeExporter commented 9 years ago
Recently, under Unix-based system, heekscad's version of tinyxml have been 
fixed to produce a library that is not in conflict with regular tinyxml. More 
work have to be done to use tinyxml from system (ie. heekscad's version is 
based on a old tinyxml version, with custom patches)

I set this issue as an enhancement, since there is no conflict anymore.

Original comment by neomil...@gmail.com on 28 Jun 2014 at 4:39