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

Refine documentation about externally maintained and unmaintained linux packages. #12507

Merged
merged 1 commit into from
Jan 17, 2022

Conversation

ekpyron
Copy link
Member

@ekpyron ekpyron commented Jan 10, 2022

Maybe not uncontroversial, but:

I don't think we should have this in the repo, if none of us actively maintains it and the external support cycle is as long as it seems to be. Is there a particular reason why this has to be in our repo? Can't people still maintain snap packages externally, if they want to, without having this in our repo?

@ekpyron
Copy link
Member Author

ekpyron commented Jan 10, 2022

If we agree to remove this, we should also remove it from the "installing the solidity compiler" docs.

The ``solc`` snap uses strict confinement. This is the most secure mode for snap packages
but it comes with limitations, like accessing only the files in your ``/home`` and ``/media`` directories.
For more information, go to `Demystifying Snap Confinement <https://snapcraft.io/blog/demystifying-snap-confinement>`_.

Arch Linux also has packages, albeit limited to the latest development version:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, I also think we should make it clear that we don't maintain the Archlinux and Gentoo ones ourselves...

@cameel
Copy link
Member

cameel commented Jan 12, 2022

Pinging @ElOpio so that he can chime in.

From #11940 looks like the current problem is not that it's broken (it's now up to date in the edge channel) but, as I understand it, it's done for the code from develop and we need to release 0.8.12 before it can hit beta and then stable. Also, there was pretty much no feedback from any interested users.

I agree that it looks like having package configuration not maintained by us in the repo did not turn out great and is likely not the way forward for other packages. But in case snap in particular it looks like things are getting under control and it'll start working with 0.8.12 so I wouldn't throw it away now. I'd vote for keeping it as long as it stays actively maintained by @ElOpio.

Is there a particular reason why this has to be in our repo? Can't people still maintain snap packages externally, if they want to, without having this in our repo?

From the original PR that added it (#2475 (comment)):

The continuous delivery in build.snapcraft.io and launchpad.net assume that the file is in {root}/snapcraft.yaml, or in {root}/snap/snapcraft.yaml. We can put it anywhere else, but we would have to use travis or something like that for CD, which is a little more work.

So doable, of course, just not as comfortable. It's your repo, so your decision. Let me know if I should move it.

@ekpyron
Copy link
Member Author

ekpyron commented Jan 12, 2022

Alright then, we can wait and if it works out again keep it in here. Closing this for the time being then.

@ekpyron ekpyron closed this Jan 12, 2022
@ekpyron
Copy link
Member Author

ekpyron commented Jan 12, 2022

We should probably still mention in the docs that the snap, archlinux and gentoo packages are externally maintained.

@come-maiz
Copy link
Contributor

I'm not using solidity or snapcraft on my job anymore, so being an active maintainer sounds a little far from my current interests. I still like both projects so I can take a look every now at the automated pipelines and then and check if things get stuck. I won't spend a lot of time testing this though. If you don't want to maintain it, instead of "externally maintained" I would call it "unmaintained". "Maintained by the robot uprising syndicate" :)

@ekpyron ekpyron reopened this Jan 13, 2022
@ekpyron ekpyron marked this pull request as ready for review January 13, 2022 09:56
@ekpyron ekpyron requested a review from cameel January 13, 2022 09:57
@cameel
Copy link
Member

cameel commented Jan 13, 2022

I think it would be a waste to just drop this completely. Maybe we could encourage some people from the community to step in and help maintain this? I mean, it does not necessarily have to sit inside the main repo but would still nice to have it somewhere. People clearly want these packages and work on snap one is mostly done.

This situation looks a bit like the one with translations where we do have some people willing to contribute but not enough so quality is a bit lacking. We could create a separate repo (or a whole org though that might be an overkill) for third-party packages and set up some rules on contribution. Then promote it a bit and only consider a package official if it reaches some level of quality and has at least two active maintainers.

@franzihei What's your opinion on this? Do you think it would be worth the effort?

@franzihei
Copy link
Member

As discussed in the chat, I think it's best to for now, not remove it entirely yet but rather say it's "unmaintained" and also add the info to relevant others (e.g. Arch) that they are externally maintained (if they are).

Then, if nobody shows interest in snap in the coming months we can still consider removing it from the docs completely. If it's the other way around and it turns out people have interest we can try to help find/coordinate external maintainers.

@ekpyron ekpyron changed the title Remove snap support. Refine documentation about externally maintained and unmaintained linux packages. Jan 13, 2022
docs/installing-solidity.rst Outdated Show resolved Hide resolved
cameel
cameel previously approved these changes Jan 14, 2022
Copy link
Member

@cameel cameel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have some tweak suggestions but overall it's fine so I'm approving already.

docs/installing-solidity.rst Outdated Show resolved Hide resolved
docs/installing-solidity.rst Outdated Show resolved Hide resolved
docs/installing-solidity.rst Outdated Show resolved Hide resolved
@come-maiz
Copy link
Contributor

Then, if nobody shows interest in snap in the coming months we can still consider removing it from the docs completely. If it's the other way around and it turns out people have interest we can try to help find/coordinate external maintainers.

Makes sense.
When I was working on this thing, we did a bunch of calls for testing. They helped a lot to get new contributors and have fun with them.
https://forum.snapcraft.io/t/call-for-testing-ipfs/97

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

Successfully merging this pull request may close these issues.

4 participants