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

A strange error related to 5 version #1108

Closed
aborruso opened this issue Oct 20, 2022 · 5 comments
Closed

A strange error related to 5 version #1108

aborruso opened this issue Oct 20, 2022 · 5 comments
Assignees
Labels

Comments

@aborruso
Copy link
Contributor

Dear @johnkerl ,
I do not know if I can open issues related to 5 version.

I have this input:

n1 q1 a1
n2 q2 a2
n3 q3 a3

If I run the command below using Miller v5.10.2

mlr --nidx label id,sequence,reply then put '$format="{\"type\":\"text\"}"' then reshape -i sequence,reply,format -o k,v input.txt

I have

free(): invalid pointer
[1]    3918 abort      mlr --nidx label id,sequence,reply then put '$format="{\"type\":\"text\"}"'

Using Miller 6 I have

n1 sequence q1
n1 reply a1
n1 format {"type":"text"}
n2 sequence q2
n2 reply a2
n2 format {"type":"text"}
n3 sequence q3
n3 reply a3
n3 format {"type":"text"}

If it no longer makes sense to open an issue for version 5, I close it.

Thank you

@johnkerl johnkerl self-assigned this Oct 20, 2022
@johnkerl johnkerl added the bug label Oct 20, 2022
@johnkerl
Copy link
Owner

@aborruso memory corruption is definitely a bug :(

I can make a branch off of 5.10.3 and put up a fix. Then I can tag a 5.10.4 release. It'll be a "pre-release" and won't be available via any package manager but you'll be able to get a binary executable for it.

johnkerl added a commit that referenced this issue Nov 26, 2022
johnkerl added a commit that referenced this issue Nov 26, 2022
@johnkerl
Copy link
Owner

@aborruso
Copy link
Contributor Author

No problem to compile Miller 5 and it works. Thank you

Note: if I run ./mlr --version I have still Miller 5.10.3

@johnkerl johnkerl reopened this Nov 27, 2022
@johnkerl
Copy link
Owner

@aborruso I recreated the tag https://github.com/johnkerl/miller/releases/tag/v5.10.4 with 5.10.4 in c/mlrvers.h -- thank you! :)

@aborruso
Copy link
Contributor Author

@aborruso I recreated the tag https://github.com/johnkerl/miller/releases/tag/v5.10.4 with 5.10.4 in c/mlrvers.h -- thank you! :)

thank you

@johnkerl johnkerl removed the active label Aug 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants