-
Notifications
You must be signed in to change notification settings - Fork 241
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
Can not download binary talisman_darwin_amd64 (v1.29.1) #401
Comments
A few members of my team and myself have had a similar problem with this issue, but it is more extreme as it is stopping us from using talisman at all. It seems related to this same issue. Downloading the above mentioned file manually from bithub fails for me in the same way, but the bigger problem is that when running talisman locally, it detects it is out of date and automatically downloads this binary as an update. The download seems to succeed from the terminal, but then when trying to run the pre-commit hooks it just breaks with a segmentation fault:
All talisman commands then fail in the same way. Here is an example of finding which talisman binary is being used and trying to execute it with another segmentation fault. I also think it might be odd that this binary is only 2.5M as previous versions have been much larger ( ~8M, I think )
This is stopping us from being able to use talisman at all, and is forcing us to remove the hooks from our repo's - so any help or guidance would be much appreciated! May be related to the recent Ventura upgrade - OS info is attached below: Machine: Mac 16 inch Intel Core i7 Happy to provide any more info that might be useful! |
It should be a bug/ compatibility issue with |
Hi @Scatchell , In our case, we have a custom installation script and the workaround was to pin the version to its previous version https://github.com/thoughtworks/talisman/releases/download/v1.29.0 We can still use Talisman on our pipeline but would definitely change the version to latest as soon as the compress bug/compatibility issue is gone. @dcRUSTy , thanks for the info! I'm using Monterey... not sure if you are referring to Ventura compatibility on the building phrase? |
@coadtantw currently macOS amd64(Intel/non M1) variant is compressed with upx
The short term fastest solution would be create a new release where https://github.com/thoughtworks/talisman/blob/main/compress-binary.sh#L14 |
@dcRUSTy Is this resolved? I tried this today and still got this same error. macOS: 13.0.1 (ventura) |
As far as I can tell, the problem is still the same. Getting the error in all repositories. Current workaround to be able to push without having talisman check the code: mv .git/hooks/pre-push .git/
git push
mv .git/pre-push .git/hooks |
thanks for reporting, i’ll take a look |
@jmatias any updates ? |
Yes, still same with me as well - have removed talisman from all repo's but from time to time I check to see if it auto updates and downloads a new working binary - checking this morning I get the usual segmentation fault
|
I'm going to debug this on an Intel Mac, I'll post an update a bit later today. @Rajat-Sharma-thoughtworks @Scatchell |
@Scatchell @Rajat-Sharma-thoughtworks Could you give it a try now? |
It worked for me as well. Thanks @jmatias |
Workaround released in version v1.29.4. https://github.com/thoughtworks/talisman/releases/tag/v1.29.4 |
Describe the bug
Can not download binary talisman_darwin_amd64 (v1.29.1) on the release page
To Reproduce
Expected behavior
It should be downloaded successfully
Screenshots
![Screen Shot 2565-10-25 at 11 22 21 AM](https://user-images.githubusercontent.com/99397204/197681958-66c75e22-c36e-40b4-8507-cd145f4adeea.png)
Desktop (please complete the following information):
Additional context
The text was updated successfully, but these errors were encountered: