Closed hayk-skydio closed 1 year ago
Issue 1 looks like it may have been fixed by @bradley-solliday-skydio in #189
Agree I think 1 should be fixed by #189 , and the other two should probably be their own issues
2) Hunch is that this is a "macOS sed
doesn't match linux sed
thing"? And that find test -name "*_codegen_test.py" | sed 's/test\/\(\w\+\).py/update_\1/g'
is empty or something
3) This is seemingly a bug in ply
that was fixed in 2013? Is your version of ply
<= 3.8 somehow?
To summarize the state of this issue, issue 1 has been fixed for some time now, and issue 2 was resolved recently (here: https://github.com/symforce-org/symforce/pull/264). Aaron's hypothesis that this had to do with the different behavior of sed
on macos was correct. Was resolved by updating sed
's arguments.
Still need to try reproducing issue 3 with an old version of ply
to confirm that that is the source of issue. If it is, I'll close the issue on the basis that I think the solution is to simply use a newer version of ply.
I don't think the ply
thing is a symforce issue - if we decide it is because someone hits it again we'll make a separate issue
Describe the bug I'm having a series of build issues that are confusing me. Perhaps these should be split issues.
To Reproduce
rm -rf build
&&pip install -ve .
Issue 1:
make lint
andmake check_types
give an error:Issue 2:
make test_update
does nothing:Issue 3:
make docs
fails:Environment (please complete the following information):
Darwin blitzer.local 21.3.0 Darwin Kernel Version 21.3.0: Wed Jan 5 21:37:58 PST 2022; root:xnu-8019.80.24~20/RELEASE_ARM64_T6000 arm64
Python 3.10.4
v0.5.0