Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

594 #28

Closed
wants to merge 3 commits into from
Closed

594 #28

wants to merge 3 commits into from

Conversation

trufae
Copy link
Contributor

@trufae trufae commented Aug 8, 2024

Checklist

  • Closing issues: #issue
  • Mark this if you consider it ready to merge
  • I've added tests (optional)
  • I wrote some documentation

Description

@sylvainpelissier
Copy link
Collaborator

Should we close this one ?

@trufae
Copy link
Contributor Author

trufae commented Aug 13, 2024

Is the ci now using latest r2-594?

@sylvainpelissier
Copy link
Collaborator

No it fails for linux20: #32

@trufae
Copy link
Contributor Author

trufae commented Aug 13, 2024

Fails because of the testsuite not because of r2

@sylvainpelissier
Copy link
Collaborator

In #32 the error is not in the testsuite it is:
/usr/bin/ld: /home/runner/.local/share/radare2/prefix/lib/libyara.a(la-threading.o): in function `yr_thread_storage_get_value': threading.c:(.text+0xf7): undefined reference to `pthread_getspecific' collect2: error: ld returned 1 exit status

@trufae
Copy link
Contributor Author

trufae commented Aug 14, 2024

Looks like missing -lpthread

@trufae trufae closed this Aug 17, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants