-
Notifications
You must be signed in to change notification settings - Fork 613
fix libgcrypt dependency on debian 8 jessie #501
Conversation
@jasonsanjose Is this change only correct after Linux has switched to the new CEF? (PR #499 hasn't been merged yet...) |
No, this shouldn't require the new CEF. However, I'm having some trouble building with CEF 1547 when merging this PR with master, so I'm unable to confirm that at the moment. |
Good morning See also thread adobe/brackets#10255 re: libcrypt version breaks Brackets As per posting in that thread from Jason, installed and tested build at https://github.com/adobe/brackets/releases/download/linux-cef-2171/Brackets.Release.1.2.64-bit.deb. Previous version of Brackets purged. libgcrypt11 purged. Installed Brackets from above link. No errors, no warnings. Runs (so far) with no problems. Debian Jessie with KDE as desktop environment. With greetings Romane |
@jasonsanjose Do you have some cycles to validate this PR? Is this good to be merged? |
Sorry, I don't have any spare cycles. |
Let me get my vm image back to work, then I can check this out. |
I saw this discussed in adobe/brackets#10255 and just want to confirm that this will not work. The .deb will install of course, but Brackets won't start. Even string-replacing all references to libgcrypt.so.11 with libgcrypt.so.20 in lib/libcef.so will not work. The current CEF has an unavoidable hard dependency on the libgcrypt11 shared library. |
@brbsix Thanks for taking some time in testing and confirming 👍 @ingorichter Thanks for your time too! We could definently use your expertise in this issue. |
Fixed by: adobe/brackets-shell#619. This pull request is now obsolete. |
See adobe/brackets#10255