Synopsis - Issue Tracker

Bug138 Editing

Title moving to boost::filesystem
Created on 2007-05-03.18:45:36 by blindvt, last changed 2007-06-06.23:11:33 by stefan.
assigned to stefan priority low
type rfe severity normal
components general platform
status open resolution
depends on superseded by
Add Comment:
File
CC List:? blindvt, hir, jsrinivasa, stefan

Messages
Author: blindvt Date: 2007-05-03.18:45:32
Hi,

Attached patch removes class Path and instead moves to
boost::filesystem.

Don't forget to 
svn rm src/Support/Path-posix.cc src/Support/Path-win32.cc \
        src/Support/Path.cc src/Support/Path.hh

The output of setup.py test is identical compared to before the patch. I
admit that i did not do further testing yet.

If some kind soul would suggests where to put factored out methods for
make_short_path(path, base)
and, perhaps, make_abs_path(path)
then i'd be glad to hear about it (i still didn't look)
Author: blindvt Date: 2007-05-05.11:40:50
Independently of bug138, I also need to pass --with-boost-filesystem= and --with-boost-wave=
to config.py to be able to pick up the correct libs.
Author: blindvt Date: 2007-05-05.11:42:45
Independently of bug138, the way how one would specify which boost_filesystem and boost_wave lib are to be used was inconsistent.

--with-boost-filesystem=boost_filesystem-43
did work while the same for wave did not work.

Attached patch
synopsis-0.9.1-bb.ax_boost_wave-fix-specifying-lib.diff
fixes this.
Author: blindvt Date: 2007-05-06.20:09:54
Updated, not yet verified patch which puts some simple wrappers into src/Support/path.{cc,hh}:
synopsis-0.9.1-bb-20070506-2200.diff

Specifically, i didn't verify matches_path() as of yet.

relies on:
$ svn mv src/aclocal.m4 src/acinclude.m4
$ apply patch
$ svn rm src/Support/P*

obsoletes:
synopsis-0.9.1-bb.20070503-1830.diff
synopsis-0.9.1-bb.ax_boost_wave-fix-specifying-lib.diff
synopsis-0.9.1-bb.config.py-boost-libnames.diff
Author: stefan Date: 2007-06-06.23:11:28
I applied the relevant portions of this patch,
slightly edited (rev:1832).
Can you please test ?

(I'm not yet sure what to do about the 
Synopsis_0_8 branch, as there we don't yet depend 
on boost at all.)
Files
File nameUploaded
synopsis-0.9.1-bb-20070506-2200.diff blindvt, 2007-05-06.20:09:56
synopsis-0.9.1-bb.20070503-1830.diff blindvt, 2007-05-03.18:45:34
synopsis-0.9.1-bb.ax_boost_wave-fix-specifying-lib.diff blindvt, 2007-05-05.11:42:47
synopsis-0.9.1-bb.config.py-boost-libnames.diff blindvt, 2007-05-05.11:40:52
History
Date User Action Args
2007-05-03 18:45:44blindvtcreate
2007-05-05 11:40:57blindvtsetfiles: + synopsis-0.9.1-bb.config.py-boost-libnames.diff
messages: + msg678
2007-05-05 11:42:52blindvtsetfiles: + synopsis-0.9.1-bb.ax_boost_wave-fix-specifying-lib.diff
messages: + msg679
2007-05-06 20:10:02blindvtsetfiles: + synopsis-0.9.1-bb-20070506-2200.diff
messages: + msg684
2007-06-06 23:11:39stefansetstatus: new
assignedto: (no value)
messages: + msg699
nosy: + stefan