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

Can compile in OpenBSD/adJ. Closes #337 #430

Merged
merged 2 commits into from
Oct 11, 2022

Conversation

vtamara
Copy link
Contributor

@vtamara vtamara commented Aug 5, 2022

No description provided.

@EmelyanenkoK
Copy link
Member

I believe 127f577 solves it?

@vtamara
Copy link
Contributor Author

vtamara commented Aug 17, 2022

I believe 127f577 solves it?

No it doesn't, because:

  1. A more recent version of abseil-cpp is required. The commit you reference updates absail-cpp to the version of Nov 3 2021. But there were fixes in abseil-cpp to be able to compile it on OpenBSD/adJ on Apr 12 2022. Compilation issues on OpenBSD/adJ 7.1a1 abseil/abseil-cpp#1153 I have been testing and using the version of Apr 14 2022 on OpenBSD/adJ.
  2. Since there is not libdl in OpenBSD/adJ a small improvement in the Cmakefile.txt is needed:
    e2ac8e6

@vtamara
Copy link
Contributor Author

vtamara commented Sep 5, 2022

Today I syncronized with master

@vtamara
Copy link
Contributor Author

vtamara commented Oct 10, 2022

Today I syncronized with master

@EmelyanenkoK EmelyanenkoK changed the base branch from master to testnet October 11, 2022 07:32
@EmelyanenkoK EmelyanenkoK merged commit f7041cb into ton-blockchain:testnet Oct 11, 2022
neodix42 added a commit to cold-test-assets/ton that referenced this pull request Nov 28, 2022
neodix42 added a commit to cold-test-assets/ton that referenced this pull request Nov 29, 2022
neodix42 added a commit to cold-test-assets/ton that referenced this pull request Nov 29, 2022
neodix42 added a commit to cold-test-assets/ton that referenced this pull request Nov 29, 2022
@vtamara
Copy link
Contributor Author

vtamara commented Dec 29, 2022

@neodix42 Hi, could you please tell me, why you reverted this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants