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

Add libstdc++6 #66

Merged
merged 1 commit into from
Apr 19, 2021
Merged

Add libstdc++6 #66

merged 1 commit into from
Apr 19, 2021

Conversation

aminya
Copy link
Contributor

@aminya aminya commented Jan 6, 2021

Hopefully, fixes #65

@aminya
Copy link
Contributor Author

aminya commented Jan 6, 2021

@popey Does this solve the issue? I am not familiar with the syntax of snap config.

I could not find the config for Ubuntu 18, but this library should be added in that config too (if missing).

@popey
Copy link
Contributor

popey commented Jan 6, 2021

Thanks very much for the pull request. I don’t have the ability to reproduce the issue right now. I have published a build of the snap incorporating your fix in a branch. Please could you install the build on a clean system, or if you have the snap already installed, refresh to this branch, and test it?

snap install atom --channel stable/fix-65

or:

snap refresh atom --channel stable/fix-65

If you’re happy with the fix, I’ll land this PR.

Thanks again!

@aminya
Copy link
Contributor Author

aminya commented Jan 15, 2021

@popey I ran the tests using this branch, and it seems to work. So, if you can merge this that would be great. I need to get user's feedback too since I am not mainly on Ubuntu.

https://github.com/atom-community/zadeh/runs/1660884233#step:9:11

@aminya
Copy link
Contributor Author

aminya commented Jan 15, 2021

@DeeDeeG Could you do a test for us? For example, see if [email protected] works on this snap Atom.

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

@aminya I'm looking over the PR and related conversations so I can see what exactly I should expect, what needs testing, and so on.

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

@aminya once I have ide-python installed, and all its requirements (https://github.com/atom-community/ide-python#requirements), how do I open a window/interface of the IDE?

@aminya
Copy link
Contributor Author

aminya commented Jan 15, 2021

@aminya once I have ide-python installed, and all its requirements (https://github.com/atom-community/ide-python#requirements), how do I open a window/interface of the IDE?

If no error similar to #65 is thrown in Atom or in the dev tools then it means it is working.

You can also try autocomplete-path package

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

I loaded the regular stable channel of the Atom snap. (I'm running a Kubuntu 20.10 LiveDVD image in VirtualBox).

With stable Atom v1.54.0 snap, I am not seeing any activation errors for these two packages.

(I don't think the person reporting that issue left us clear steps to reproduce, so I'm not sure what problem they were seeing. It almost sounds like they had a really old OS/ old libc, installed a newer OS over it but preserving their home directory... so in their .atom folder they had pre-compiled packages with naive binaries compiled for an older system, but trying to run these old binaries on a new system. I bet if they uninstalled and reinstalled the ide-python Atom package, the error message would go away... Maybe? Who knows? As I said, the background info (not a lot) and steps to reproduce (none that we can reproduce at the moment) were not great.)

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

On the positive side: I think their issue is not due to the package or the snap, so it's likely not our responsibility to fix.

Probably the greatest amount of end-user trouble and confusion in the Atom world comes from natively built package binaries that were built against some other target than they are being initialized on...

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

I had a bunch of nifty language-aware autocomplete suggestions when editing a new file app.py. So I think it was working.

@aminya
Copy link
Contributor Author

aminya commented Jan 15, 2021

@popey this PR is ready to merge

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

@aminya it's working for me on the stable branch of the atom snap. I never even got to testing the patched version from this PR because I couldn't reproduce the initial error.

I think my testing implies this PR isn't needed.

@aminya
Copy link
Contributor Author

aminya commented Jan 15, 2021

Maybe you already have the library installed on your machine.

I still think this PR should be merged.

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

The person who reported the initial issue said they were "distro hopping" which means installing lots of different Linux OSes. I think they must have installed OSes with two different versions of libc. They built this package on one of the OSes, then tried to run it again on the second OS. That seems like a plausible explanation for the error they saw.

I really don't see a way that the Atom snap is related to that issue.

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

I guess I can try on Ubuntu 20.04.1 instead of Ubuntu 20.10...

@aminya
Copy link
Contributor Author

aminya commented Jan 15, 2021

This issue was reported 13 times by 13 different people. See all the duplicates I closed. The fact that it does not happen for you does not mean that this library is not needed.

@DeeDeeG
Copy link

DeeDeeG commented Jan 15, 2021

Well I'm also unable to reproduce this on kubuntu 20.04.1, which (20.04.1 in general) is what all the other reports were from.

Sadly I can't test the Atom version they all reported this from, Atom 1.53.0, as the newer version is out 1.54.0, and snap won't let me install the old version unless I have developer access (uploading rights?) to the Atom snap.

I have no idea if this fix is needed or not, and unless someone gives me steps that work for me to actually reproduce the issue it will be unclear to me whether this fix is relevant. I can't say either way based on any evidence or testing.

@aminya
Copy link
Contributor Author

aminya commented Feb 7, 2021

@popey Could you merge this? I have been waiting for it for a long time now.

@aminya
Copy link
Contributor Author

aminya commented Feb 18, 2021

@popey @snapcrafters @flexiondotorg could you merge this?

@aminya
Copy link
Contributor Author

aminya commented Apr 18, 2021

Anyone here?

@popey popey merged commit f24c6a5 into snapcrafters:master Apr 19, 2021
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.

Add GLIBCXX_3.4.22 to the base libraries
3 participants