Opened 17 years ago
Closed 17 years ago
#312 closed defect (fixed)
PostGIS provider : remove boost_date_time dll dependency
Reported by: | bscott | Owned by: | mloskot |
---|---|---|---|
Priority: | minor | Milestone: | 3.4.0 |
Component: | PostGIS Provider | Version: | 3.2.0 |
Severity: | 4 | Keywords: | |
Cc: | External ID: |
Description
This is the only provider that need the boost_date_time dll Or we add this dll in the distribution of FDO 3.3 and higher Or we remove this dependency, quite easy to replace by generic c++ function
ACtually i have already did the implementation patch will follow
Change History (6)
comment:1 by , 17 years ago
follow-up: 4 comment:3 by , 17 years ago
Agreed with Mateusz here. I believe that the date time library is already being distributed with FDO 3.3.x and with MapGuide 2.0.
In this case, it may have been needed to provide a granular-enough random seed for a unique session ID. Using seconds (or something slightly smaller?) turned out not to be granular enough when generating tiles at a rapid rate through multiple active connections. Transaction identifiers (or something like that) ran into naming collisions.
comment:4 by , 17 years ago
Replying to jbirch: When we install Mapguide 2.0 and Map 2009 the only boost dll present is boost_thread-vc80-mt-1_34_1.dll. the boost_date_time-vc80-mt-1_34_1.dll is not part of the install. For a end user this dll is very difficult to find. Adding this dll to the provider binaries is not a clean solution. As Map 2009 is already shiiping, it is not possible to have this dll in the setup. This is why i tough it was a simpler solution to remove this dependency.
comment:5 by , 17 years ago
Status: | new → assigned |
---|
Just to remind, I'm completely OK with removing the Boost dependency for this purpose :-)
comment:6 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed in trunk (r3904) with patch from Bruno Scott. I'm closing it as fixed. Please, reopen if something is still not working well.
Bruno,
I don't mind to remove the dependency, however I'd add that we've agreed to allow use of any library included in the Boost set.