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

parse issue $PROJECT #865

Open
asg0451 opened this issue Nov 18, 2023 · 9 comments
Open

parse issue $PROJECT #865

asg0451 opened this issue Nov 18, 2023 · 9 comments
Labels
bug Something isn't working

Comments

@asg0451
Copy link

asg0451 commented Nov 18, 2023

❯ kustomize
^^ type x to run that

❯ x
env +kustomize && kustomize
× parse error unidata.ucar.edu/netcdf
│ this is a serious issue. please report the bug
╰─➤ https://github.com/pkgxdev/pkgx/issues/new?title=parse+issue+unidata.ucar.edu/netcdf

kitty, zsh, mac os sonoma 14.1.1

worked on a retry

@mxcl
Copy link
Member

mxcl commented Nov 18, 2023

oh dang, the retry presumably fixed the corrupted file. However we are getting more of these sorts of reports than I'm comfortable with so the bug is there. I wonder specifically what is going wrong.

@trent-medici
Copy link

Same here, which just happened after updating my Mac M1 to Sonoma 14.2 today. (It worked yesterday before updating.)

$ pkgx +npmjs.com sh
× parse error github.com/skystrife/cpptoml
│ this is a serious issue. please report the bug
╰─➤ https://github.com/pkgxdev/pkgx/issues/new?title=parse+issue+github.com/skystrife/cpptoml


$ sh <(curl https://pkgx.sh) +npmjs.com sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3764  100  3764    0     0   7058      0 --:--:-- --:--:-- --:--:--  7061
#################################################################################################################### 100.0%
× parse error github.com/skystrife/cpptoml
│ this is a serious issue. please report the bug
╰─➤ https://github.com/pkgxdev/pkgx/issues/new?title=parse+issue+github.com/skystrife/cpptoml


$ x
zsh: command not found: x


# works with no args
$ pkgx sh
this is a temporary shell containing the following packages:

when done type: `exit`
pkgx ~/dev/medici/las-web/web


$ pkgx --version
pkgx 1.0.4

Mine is bad because it does not work on a retry, nor with x. Any workarounds appreciated!

@jhheider
Copy link
Contributor

pkgx --sync should force fix it. can you paste the contents of your ~/Library/Application\ Support/pkgx/pantry/projects/github.com/skystrife/cpptoml/package.yml first?

@trent-medici
Copy link

pkgx --sync should force fix it. can you paste the contents of your ~/Library/Application\ Support/pkgx/pantry/projects/github.com/skystrife/cpptoml/package.yml first?

$ cat ~/Library/Application\ Support/pkgx/pantry/projects/github.com/skystrife/cpptoml/package.yml
$ ls -al ~/Library/Application\ Support/pkgx/pantry/projects/github.com/skystrife/cpptoml/package.yml
-rw-r--r--  1 tlarson  staff  0 Dec 12 13:17 /Users/tlarson/Library/Application Support/pkgx/pantry/projects/github.com/skystrife/cpptoml/package.yml

@jhheider
Copy link
Contributor

wow. empty.

@trent-medici
Copy link

Interesting that it is the ONLY file in the 724 files under that github.com directory that is size 0.

$ ls -l  ~/Library/Application\ Support/pkgx/pantry/projects/github.com/*/* | grep " 0 "
-rw-r--r--  1 tlarson  staff    0 Dec 12 13:17 package.yml

@trent-medici
Copy link

pkgx --sync

That worked! Thank you very much.

@jhheider
Copy link
Contributor

There should be over 1000 package.yml files alone. I'm guessing it was the last thing written during the unpack of a truncated tarballs.

My pet theory is that github's download repo as tarball isn't working 100%. Either because it's reusing temp files or because we have so many files.

mxcl added a commit that referenced this issue Jan 5, 2024
Will fix some errors due to invalid pantry parsing

Refs #882
Refs #865
mxcl added a commit that referenced this issue Jan 5, 2024
Will fix some errors due to invalid pantry parsing

Refs #882
Refs #865
mxcl added a commit that referenced this issue Jan 5, 2024
* Will fix some errors due to invalid pantry parsing
* Removes platform specific moustaches devenvs, these weren't documented so even though this is strictly a breaking change I would be willing to risk the tea protocol slash consequences for it in a tea protocol future

Refs #882
Refs #865
mxcl added a commit that referenced this issue Jan 5, 2024
* Will fix some errors due to invalid pantry parsing
* Removes platform specific moustaches devenvs
  * these weren't documented so even though this is strictly a breaking change I would be willing to risk the tea protocol slash consequences for it in a tea protocol future
  * there is potential utility in platform/arch being there so we should propose a proper addition
* Fix CD

Refs #882
Refs #865
mxcl added a commit that referenced this issue Jan 5, 2024
* Will fix some errors due to invalid pantry parsing
* Removes platform specific moustaches devenvs
  * these weren't documented so even though this is strictly a breaking change I would be willing to risk the tea protocol slash consequences for it in a tea protocol future
  * there is potential utility in platform/arch being there so we should propose a proper addition
* Fix CD

Refs #882
Refs #865
@mxcl mxcl changed the title parse issue unidata.ucar.edu/netcdf parse issue ******** Feb 2, 2024
@mxcl mxcl pinned this issue Feb 2, 2024
@mxcl mxcl added the bug Something isn't working label Feb 2, 2024
@mxcl
Copy link
Member

mxcl commented Feb 2, 2024

This should effect things less often now, but still periodically happens for sure. I have plans for solutions based on the precept that this is a github tarball corruption issue.

@mxcl mxcl changed the title parse issue ******** parse issue $PROJECT Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants