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

Import Version 1.7 #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Import Version 1.7 #2

wants to merge 2 commits into from

Conversation

the-reverend
Copy link

Per our conversation, this is the diff for version 1.7 from neilb-csvfix-93804d497be6.zip. I tested the build on WSL. I don't have a build environment set up for Mac or Win64. In order to run the test suite in linux, I had to change the line endings on all the shell scripts, fix the path to bash, and change the line endings to all the test files and data, but I did not commit those adjustments because I didn't have time to make it cross-platform. A few tests are failing, but these also failed in version 1.6.

@wlbr
Copy link
Owner

wlbr commented Aug 8, 2023

Sorry, I don't have the build environments as well. I used the Github CI for that.
So you could copy my Gitub Action to your repo and make the compile runs work. Then we could merge it into the repo here.

Sorry, I don't have the time to fix that here.

@AndrewSav
Copy link

@the-reverend I merged it here. And a test build is available here. It seems to work fine on Windows / Linux and I have no access to mac so I cannot test that. If you would like to move this forward in any shape or form please let me know what you think.

@the-reverend
Copy link
Author

hey thanks @AndrewSav! sorry it took so long for me to notice your merge. i'll see what i can do on my wife's macbook.

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.

3 participants