Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/master/ofx2qif #448

Closed
wants to merge 11 commits into from

Conversation

loki36
Copy link
Contributor

@loki36 loki36 commented Apr 11, 2015

add convert ofx to qif under linux using ofx2qif command depending of ofx package
#300

@gabriele-v
Copy link
Contributor

If it works fine, what about insert ofxtoqif as MMEX 3rd library?

@loki36
Copy link
Contributor Author

loki36 commented Apr 12, 2015

I think it's the best and easy way but for this we need to find the compile version for windows and for mac

@vomikan
Copy link
Member

vomikan commented Apr 13, 2015

Please make a patch for this changes and apply it for new branch.
Remove white spaces and tabs.
Then provide another PR.
Thanks.

@loki36
Copy link
Contributor Author

loki36 commented Apr 13, 2015

ok
I will do it asap

@vomikan
Copy link
Member

vomikan commented Apr 16, 2015

It's bad style to use OS specific code. On other hand there are no open ofx2qif converters writen on c++.

@guanlisheng
Copy link
Contributor

i have the similar concern as Nikolay and would we implement a general and cross platform one?

LiSheng

在 2015年4月16日,下午10:21,Nikolay notifications@github.com 写道:

It's bad style to use OS specific code. On other hand there are no open ofx2qif converters writen on c++.


Reply to this email directly or view it on GitHub.

@ghost
Copy link

ghost commented Apr 18, 2015

According to libofx homepage, it is written in C++

http://libofx.sourceforge.net/

@vomikan
Copy link
Member

vomikan commented Apr 18, 2015

The same https://github.com/arolson101/libofx ?

2015-04-18 23:28 GMT+03:00 David Jones notifications@github.com:

According to libofx homepage, it is written in C++

http://libofx.sourceforge.net/


Reply to this email directly or view it on GitHub
#448 (comment)
.

@ghost
Copy link

ghost commented Apr 20, 2015

I think so

@vomikan
Copy link
Member

vomikan commented Apr 23, 2015

Lets close this PR.
Try to add https://github.com/arolson101/libofx as submodule and use it for all systems.

@vomikan vomikan closed this May 11, 2015
@arthurp arthurp mentioned this pull request Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants