diff --git a/pkgs/development/libraries/libofx/default.nix b/pkgs/development/libraries/libofx/default.nix
new file mode 100644
index 000000000000..2043381ffc6f
--- /dev/null
+++ b/pkgs/development/libraries/libofx/default.nix
@@ -0,0 +1,22 @@
+{ stdenv, fetchurl, opensp, pkgconfig, libxml2, curl }:
+        
+stdenv.mkDerivation rec {
+  name = "libofx-0.9.1";
+
+  src = fetchurl {
+    url = "mirror://sourceforge/libofx/${name}.tar.gz";
+    sha256 = "0gyana7v3pcqdpncjr5vg5z2r2z3rvg0fiml59mazi9n62zk86rj";
+  };
+
+  patches = [ ./libofx-0.9.0-gcc43.patch ];
+
+  configureFlags = [ "--with-opensp-includes=${opensp}/include/OpenSP" ];
+  buildInputs = [ opensp pkgconfig libxml2 curl ];
+
+  meta = { 
+    description = "Opensource implementation of the Open Financial eXchange specification";
+    homepage = http://libofx.sourceforge.net/;
+    license = "LGPL";
+  };
+}
+
diff --git a/pkgs/development/libraries/libofx/libofx-0.9.0-gcc43.patch b/pkgs/development/libraries/libofx/libofx-0.9.0-gcc43.patch
new file mode 100644
index 000000000000..63ced2d5f81c
--- /dev/null
+++ b/pkgs/development/libraries/libofx/libofx-0.9.0-gcc43.patch
@@ -0,0 +1,375 @@
+http://bugs.gentoo.org/218782
+diff -NrU5 libofx-0.9.0.orig/lib/ofx_container_transaction.cpp libofx-0.9.0/lib/ofx_container_transaction.cpp
+--- libofx-0.9.0.orig/lib/ofx_container_transaction.cpp	2007-11-10 00:10:49.000000000 +0100
++++ libofx-0.9.0/lib/ofx_container_transaction.cpp	2008-04-21 22:26:24.000000000 +0200
+@@ -19,10 +19,11 @@
+ 
+ #ifdef HAVE_CONFIG_H
+ #include <config.h>
+ #endif
+ 
++#include <cstdlib>
+ #include <string>
+ #include "messages.hh"
+ #include "libofx.h"
+ #include "ofx_containers.hh"
+ #include "ofx_utilities.hh"
+diff -NrU5 libofx-0.9.0.orig/lib/ofx_request_accountinfo.cpp libofx-0.9.0/lib/ofx_request_accountinfo.cpp
+--- libofx-0.9.0.orig/lib/ofx_request_accountinfo.cpp	2007-10-28 03:33:43.000000000 +0100
++++ libofx-0.9.0/lib/ofx_request_accountinfo.cpp	2008-04-21 22:26:24.000000000 +0200
+@@ -19,10 +19,11 @@
+ 
+ #ifdef HAVE_CONFIG_H
+ #include <config.h>
+ #endif
+ 
++#include <cstdlib>
+ #include <string>
+ #include "libofx.h"
+ #include "ofx_request_accountinfo.hh"
+ 
+ using namespace std;
+diff -NrU5 libofx-0.9.0.orig/lib/ofx_request.cpp libofx-0.9.0/lib/ofx_request.cpp
+--- libofx-0.9.0.orig/lib/ofx_request.cpp	2007-10-28 03:37:52.000000000 +0100
++++ libofx-0.9.0/lib/ofx_request.cpp	2008-04-21 22:26:24.000000000 +0200
+@@ -19,10 +19,11 @@
+ 
+ #ifdef HAVE_CONFIG_H
+ #include <config.h>
+ #endif
+ 
++#include <cstring>
+ #include <string>
+ #include "messages.hh"
+ #include "libofx.h"
+ #include "ofx_request.hh"
+ 
+diff -NrU5 libofx-0.9.0.orig/lib/ofx_request_statement.cpp libofx-0.9.0/lib/ofx_request_statement.cpp
+--- libofx-0.9.0.orig/lib/ofx_request_statement.cpp	2007-10-28 13:57:18.000000000 +0100
++++ libofx-0.9.0/lib/ofx_request_statement.cpp	2008-04-21 22:26:24.000000000 +0200
+@@ -19,10 +19,11 @@
+ 
+ #ifdef HAVE_CONFIG_H
+ #include <config.h>
+ #endif
+ 
++#include <cstdlib>
+ #include <string>
+ #include "libofx.h"
+ #include "ofx_utilities.hh"
+ #include "ofx_request_statement.hh"
+ 
+diff -NrU5 libofx-0.9.0.orig/lib/ofx_utilities.cpp libofx-0.9.0/lib/ofx_utilities.cpp
+--- libofx-0.9.0.orig/lib/ofx_utilities.cpp	2007-10-27 14:12:49.000000000 +0200
++++ libofx-0.9.0/lib/ofx_utilities.cpp	2008-04-21 22:26:24.000000000 +0200
+@@ -19,11 +19,12 @@
+ #include <iostream>
+ #include <assert.h>
+ 
+ #include "ParserEventGeneratorKit.h"
+ #include "SGMLApplication.h"
+-#include <time.h>
++#include <ctime>
++#include <cstdlib>
+ #include <string>
+ #include <locale.h>
+ #include "messages.hh"
+ #include "ofx_utilities.hh"
+ 
+diff -NrU5 libofx-0.9.0.orig/lib/ofx_utilities.cpp.orig libofx-0.9.0/lib/ofx_utilities.cpp.orig
+--- libofx-0.9.0.orig/lib/ofx_utilities.cpp.orig	1970-01-01 01:00:00.000000000 +0100
++++ libofx-0.9.0/lib/ofx_utilities.cpp.orig	2007-10-27 14:12:49.000000000 +0200
+@@ -0,0 +1,245 @@
++/***************************************************************************
++                          ofx_util.cpp
++                             -------------------
++    copyright            : (C) 2002 by Benoit Grégoire
++    email                : bock@step.polymtl.ca
++ ***************************************************************************/
++/**@file
++ * \brief Various simple functions for type conversion & al
++ */
++/***************************************************************************
++ *                                                                         *
++ *   This program is free software; you can redistribute it and/or modify  *
++ *   it under the terms of the GNU General Public License as published by  *
++ *   the Free Software Foundation; either version 2 of the License, or     *
++ *   (at your option) any later version.                                   *
++ *                                                                         *
++ ***************************************************************************/
++#include <config.h>
++#include <iostream>
++#include <assert.h>
++
++#include "ParserEventGeneratorKit.h"
++#include "SGMLApplication.h"
++#include <time.h>
++#include <string>
++#include <locale.h>
++#include "messages.hh"
++#include "ofx_utilities.hh"
++
++#ifdef OS_WIN32
++# define DIRSEP "\\"
++#else
++# define DIRSEP "/"
++#endif
++
++
++using namespace std;
++/**
++   Convert an OpenSP CharString directly to a C++ stream, to enable the use of cout directly for debugging.
++*/ 
++/*ostream &operator<<(ostream &os, SGMLApplication::CharString s)
++  {
++  for (size_t i = 0; i < s.len; i++)
++  {
++  os << ((char *)(s.ptr))[i*sizeof(SGMLApplication::Char)];
++  }
++  return os;
++  }*/
++
++/*wostream &operator<<(wostream &os, SGMLApplication::CharString s)
++  {
++  for (size_t i = 0; i < s.len; i++)
++  {//cout<<i;
++  os << wchar_t(s.ptr[i*MULTIPLY4]);  
++  }
++  return os;
++  }            */
++
++/*wchar_t* CharStringtowchar_t(SGMLApplication::CharString source, wchar_t *dest)
++  {
++  size_t i;
++  for (i = 0; i < source.len; i++)
++  {
++  dest[i]+=wchar_t(source.ptr[i*sizeof(SGMLApplication::Char)*(sizeof(char)/sizeof(wchar_t))]);
++  }
++  return dest;
++  }*/
++
++string CharStringtostring(const SGMLApplication::CharString source, string &dest)
++{
++  size_t i;
++  dest.assign("");//Empty the provided string
++  //  cout<<"Length: "<<source.len<<"sizeof(Char)"<<sizeof(SGMLApplication::Char)<<endl;
++  for (i = 0; i < source.len; i++){
++    dest+=(char)(((source.ptr)[i]));  
++    //    cout<<i<<" "<<(char)(((source.ptr)[i]))<<endl; 
++  }
++  return dest;
++}
++
++string AppendCharStringtostring(const SGMLApplication::CharString source, string &dest)
++{
++  size_t i;
++  for (i = 0; i < source.len; i++)
++    {
++      dest+=(char)(((source.ptr)[i]));
++    }
++  return dest;
++}
++
++/** 
++ * Converts a date from the YYYYMMDDHHMMSS.XXX[gmt offset:tz name] OFX format (see OFX 2.01 spec p.66) to a C time_t.
++ * @param ofxdate date from the YYYYMMDDHHMMSS.XXX[gmt offset:tz name] OFX format
++ * @return C time_t in the local time zone
++ * @note
++ * @li The library always returns the time in the systems local time
++ * @li OFX defines the date up to the millisecond.  The library ignores those milliseconds, since ANSI C does not handle such precision cleanly.  The date provided by LibOFX is precise to the second, assuming that information this precise was provided in the ofx file.  So you wont know the millisecond you were ruined...
++ 
++ * @note DEVIATION FROM THE SPECS : The OFX specifications (both version 1.6 and 2.02) state that a client should assume that if the server returns a date without à specific time, we assume it means 0h00 GMT.  As such, when we apply the local timezone and for example you are in the EST timezone, we will remove 5h, and the transaction will have occurred on the prior day!  This is probably not what the bank intended (and will lead to systematic errors), but the spec is quite explicit in this respect (Ref:  OFX 2.01 spec pp. 66-68)<BR><BR>
++ * To solve this problem (since usually a time error is relatively unimportant, but date error is), and to avoid problems in Australia caused by the behaviour in libofx up to 0.6.4, it was decided starting with 0.6.5 to use the following behavior:<BR><BR>
++ * -No specific time is given in the file (date only):  Considering that most banks seem to be sending dates in this format represented as local time (not compliant with the specs), the transaction is assumed to have occurred 11h59 (just before noon) LOCAL TIME.  This way, we should never change the date, since you'd have to travel in a timezone at least 11 hours backwards or 13 hours forward from your own to introduce mistakes.  However, if you are in timezone +13 or +14, and your bank meant the data to be interpreted by the spec, you will get the wrong date.  We hope that banks in those timezone will either represent in local time like most, or specify the timezone properly.<BR><BR>
++ * -No timezone is specified, but exact time is, the same behavior is mostly used, as many banks just append zeros instead of using the short notation.  However, the time specified is used, even if 0 (midnight).<BR><BR>
++ * -When a timezone is specified, it is always used to properly convert in local time, following the spec.
++ *
++ */
++time_t ofxdate_to_time_t(const string ofxdate)
++{
++  struct tm time;
++  double local_offset; /* in seconds */
++  float ofx_gmt_offset; /* in fractionnal hours */
++  char timezone[4]; /* Original timezone: the library does not expose this value*/
++  char exact_time_specified = false;
++  char time_zone_specified = false;
++
++  time_t temptime;
++  std::time(&temptime);
++  local_offset = difftime(mktime(localtime(&temptime)), mktime(gmtime(&temptime)));
++  
++  if(ofxdate.size()!=0){
++    time.tm_year=atoi(ofxdate.substr(0,4).c_str())-1900;
++    time.tm_mon=atoi(ofxdate.substr(4,2).c_str())-1;
++    time.tm_mday=atoi(ofxdate.substr(6,2).c_str());
++    if(ofxdate.size()>8) {
++    /* if exact time is specified */
++exact_time_specified = true;
++      time.tm_hour=atoi(ofxdate.substr(8,2).c_str());
++      time.tm_min=atoi(ofxdate.substr(10,2).c_str());
++      time.tm_sec=atoi(ofxdate.substr(12,2).c_str());
++    }
++    
++    /* Check if the timezone has been specified */
++    string::size_type startidx = ofxdate.find("[");
++    string::size_type endidx;
++    if(startidx!=string::npos){
++      /* Time zone was specified */
++      time_zone_specified = true;
++      startidx++;
++      endidx = ofxdate.find(":", startidx)-1;
++      ofx_gmt_offset=atof(ofxdate.substr(startidx,(endidx-startidx)+1).c_str());
++      startidx = endidx+2;
++      strncpy(timezone,ofxdate.substr(startidx,3).c_str(),4);
++    }
++    else{
++      /* Time zone was not specified, assume GMT (provisionnaly) in case exact time is specified */
++      ofx_gmt_offset=0;
++      strcpy(timezone, "GMT");
++    }
++
++    if(time_zone_specified == true)
++      {
++	/* If the timezone is specified always correct the timezone */
++	/* If the timezone is not specified, but the exact time is, correct the timezone, assuming GMT following the spec */
++	/* Correct the time for the timezone */
++	time.tm_sec = time.tm_sec + (int)(local_offset - (ofx_gmt_offset*60*60));//Convert from fractionnal hours to seconds
++      }
++    else if (exact_time_specified == false)
++      {
++	/*Time zone data missing and exact time not specified, diverge from the OFX spec ans assume 11h59 local time */
++       time.tm_hour=11;
++       time.tm_min=59;
++       time.tm_sec=0;
++      }
++  }
++  else{
++    message_out(ERROR, "ofxdate_to_time_t():  Unable to convert time, string is 0 length!");
++  }
++  return mktime(&time);
++}
++
++/**
++ * Convert a C++ string containing an amount of money as specified by the OFX standard and convert it to a double float.
++ *\note The ofx number format is the following:  "." or "," as decimal separator, NO thousands separator.
++ */ 
++double ofxamount_to_double(const string ofxamount)
++{
++  //Replace commas and decimal points for atof()
++  string::size_type idx;
++  string tmp = ofxamount;
++
++  idx = tmp.find(',');
++  if(idx==string::npos){
++    idx = tmp.find('.');
++  }
++  
++  if(idx!=string::npos){
++    tmp.replace(idx,1,1,((localeconv())->decimal_point)[0]);
++  }
++
++  return atof(tmp.c_str());
++}
++
++/**
++Many weird caracters can be present inside a SGML element, as a result on the transfer protocol, or for any reason.  This function greatly enhances the reliability of the library by zapping those gremlins (backspace,formfeed,newline,carriage return, horizontal and vertical tabs) as well as removing whitespace at the begining and end of the string.  Otherwise, many problems will occur during stringmatching.
++*/
++string strip_whitespace(const string para_string)
++{
++  size_t index;
++  size_t i;
++  string temp_string = para_string;
++  const char *whitespace = " \b\f\n\r\t\v";
++  const char *abnormal_whitespace = "\b\f\n\r\t\v";//backspace,formfeed,newline,cariage return, horizontal and vertical tabs
++  message_out(DEBUG4,"strip_whitespace() Before: |"+temp_string+"|");
++  for(i=0;i<=temp_string.size()&&temp_string.find_first_of(whitespace, i)==i&&temp_string.find_first_of(whitespace, i)!=string::npos;i++);
++  temp_string.erase(0,i);//Strip leading whitespace
++  for(i=temp_string.size()-1;(i>=0)&&(temp_string.find_last_of(whitespace, i)==i)&&(temp_string.find_last_of(whitespace, i)!=string::npos);i--);
++  temp_string.erase(i+1,temp_string.size()-(i+1));//Strip trailing whitespace
++  
++while ((index = temp_string.find_first_of(abnormal_whitespace))!=string::npos)
++  {
++    temp_string.erase(index,1);//Strip leading whitespace
++  };
++ 
++ message_out(DEBUG4,"strip_whitespace() After:  |"+temp_string+"|");
++ 
++ return temp_string;
++}
++
++
++int mkTempFileName(const char *tmpl, char *buffer, unsigned int size) {
++  const char *tmp_dir;
++
++  tmp_dir = getenv ("TMPDIR");
++  if (!tmp_dir)
++    tmp_dir = getenv ("TMP");
++  if (!tmp_dir)
++    tmp_dir = getenv ("TEMP");
++
++  if (!tmp_dir)
++    {
++#ifdef OS_WIN32
++      tmp_dir = "C:\\";
++#else  
++      tmp_dir = "/tmp";
++#endif	/* !OS_WIN32 */
++    }
++
++  strncpy(buffer, tmp_dir, size);
++  assert((strlen(buffer)+strlen(tmpl)+2)<size);
++  strcat(buffer, DIRSEP);
++  strcat(buffer, tmpl);
++  return 0;
++}
++
++
++
+diff -NrU5 libofx-0.9.0.orig/ofxconnect/ofxconnect.cpp libofx-0.9.0/ofxconnect/ofxconnect.cpp
+--- libofx-0.9.0.orig/ofxconnect/ofxconnect.cpp	2007-10-26 20:24:10.000000000 +0200
++++ libofx-0.9.0/ofxconnect/ofxconnect.cpp	2008-04-21 22:31:18.000000000 +0200
+@@ -34,10 +34,11 @@
+ #include "libofx.h"
+ #include <config.h>		/* Include config constants, e.g., VERSION TF */
+ #include <stdio.h>
+ #include <stdlib.h>
+ #include <unistd.h>
++#include <cstring>
+ 
+ #ifdef HAVE_LIBCURL
+ #include <curl/curl.h>
+ #endif
+ 
+diff -NrU5 libofx-0.9.0.orig/ofxconnect/ofxpartner.cpp libofx-0.9.0/ofxconnect/ofxpartner.cpp
+--- libofx-0.9.0.orig/ofxconnect/ofxpartner.cpp	2006-03-26 06:49:54.000000000 +0200
++++ libofx-0.9.0/ofxconnect/ofxpartner.cpp	2008-04-21 22:28:26.000000000 +0200
+@@ -33,10 +33,13 @@
+ #include <sys/stat.h>
+ 
+ #include <iostream>
+ #include <string>
+ #include <vector>
++#include <algorithm>
++#include <cstring>
++
+ 
+ using std::string;
+ using std::vector;
+ using std::cout;
+ using std::endl;
+diff -NrU5 libofx-0.9.0.orig/ofxdump/ofxdump.cpp libofx-0.9.0/ofxdump/ofxdump.cpp
+--- libofx-0.9.0.orig/ofxdump/ofxdump.cpp	2006-08-02 06:19:22.000000000 +0200
++++ libofx-0.9.0/ofxdump/ofxdump.cpp	2008-04-21 22:26:24.000000000 +0200
+@@ -27,10 +27,12 @@
+  *   (at your option) any later version.                                   *
+  *                                                                         *
+  ***************************************************************************/
+ #include <iostream>
+ #include <iomanip>
++#include <cstdlib>
++#include <cstring>
+ #include <string>
+ #include "libofx.h"
+ #include <stdio.h>		/* for printf() */
+ #include <config.h>		/* Include config constants, e.g., VERSION TF */
+ #include <errno.h>
diff --git a/pkgs/tools/text/sgml/opensp/default.nix b/pkgs/tools/text/sgml/opensp/default.nix
index 4acf13af6fbd..76efbd0b1af6 100644
--- a/pkgs/tools/text/sgml/opensp/default.nix
+++ b/pkgs/tools/text/sgml/opensp/default.nix
@@ -1,16 +1,20 @@
-{stdenv, fetchurl}:
+{stdenv, fetchurl, xmlto, docbook_xml_dtd_412, libxslt, docbook_xsl}:
 
 stdenv.mkDerivation {
-  # OpenSP-1.5.1 requires gcc 3.3 to build.
-  # The next release is likely to be compatible with newer gccs.
-  # If so the overrideGCC in top-level/all-packages should be removed.
-  name = "OpenSP-1.5.1";
+  name = "opensp-1.5.2";
 
   src = fetchurl {
-    url = "http://prdownloads.sourceforge.net/openjade/OpenSP-1.5.1.tar.gz";
-    sha256 = "0svkgk85m6f848fi3nxnrkzg62422wxr739w5r1yrmn31n24j1iz";
+    url = mirror://sourceforge/openjade/OpenSP-1.5.2.tar.gz;
+    sha256 = "1khpasr6l0a8nfz6kcf3s81vgdab8fm2dj291n5r2s53k228kx2p";
   };
 
+  patchPhase = ''
+    sed -i s,/usr/share/sgml/docbook/xml-dtd-4.1.2/,${docbook_xml_dtd_412}/xml/dtd/docbook/, \
+      docsrc/*.xml
+  '';
+
+  buildInputs = [ xmlto docbook_xml_dtd_412 libxslt docbook_xsl ];
+
   meta = {
     description = "A suite of SGML/XML processing tools";
     license = "BSD";
diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix
index b828b3f961df..68d7c8f04ff6 100644
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -1214,8 +1214,8 @@ let
   };
 
   opensp = import ../tools/text/sgml/opensp {
-    inherit fetchurl;
-    stdenv = overrideGCC stdenv gcc33;
+    inherit fetchurl xmlto docbook_xml_dtd_412 libxslt docbook_xsl;
+    inherit stdenv;
   };
 
   openvpn = import ../tools/networking/openvpn {
@@ -4117,6 +4117,10 @@ let
     inherit fetchurl stdenv;
   };
 
+  libofx = import ../development/libraries/libofx {
+    inherit fetchurl stdenv opensp pkgconfig libxml2 curl;
+  };
+
   libogg = import ../development/libraries/libogg {
     inherit fetchurl stdenv;
   };