Message ID | 20240112150546.13891-1-brgl@bgdev.pl |
---|---|
State | Superseded |
Headers | show |
Series | [libgpiod] README: add info about the github page | expand |
On Fri, Jan 12, 2024 at 04:05:46PM +0100, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > The github page over at https://github.com/brgl/libgpiod has been reopened > for bug reports and discussions. Add a link and a mention to the README > file. > No problem with the patch, but the github page itself seems to have Issues and Discussions disabled. And the Wiki might be useful too? Reviewed-by: Kent Gibson <warthog618@gmail.com> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > --- > README | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/README b/README > index 69128dc..41237f4 100644 > --- a/README > +++ b/README > @@ -294,9 +294,14 @@ Those also provide examples of the expected formatting. > Allow some time for your e-mail to propagate to the list before retrying, > particularly if there are no e-mails in the list more recent than yours. > > +There is a libgpiod github page[7] available for reporting bugs and general > +discussions and although PRs can be submitted and discussed, upstreambound > +patches need to go through the mailing list nevertheless. > + > [1] https://github.com/kward/shunit2 > [2] http://vger.kernel.org/vger-lists.html#linux-gpio > [3] https://docs.kernel.org/process/email-clients.html > [4] https://docs.kernel.org/process/coding-style.html > [5] https://docs.kernel.org/process/submitting-patches.html > [6] https://lore.kernel.org/linux-gpio/ > +[7] https://github.com/brgl/libgpiod > -- > 2.40.1 >
On Fri, Jan 12, 2024 at 11:25:02PM +0800, Kent Gibson wrote: > On Fri, Jan 12, 2024 at 04:05:46PM +0100, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > The github page over at https://github.com/brgl/libgpiod has been reopened > > for bug reports and discussions. Add a link and a mention to the README > > file. > > > > No problem with the patch, but the github page itself seems to have > Issues and Discussions disabled. And the Wiki might be useful too? > The Releases section could use updating - it lists the latest as v0.3 from June 2017! And unless you intend to handle everything that lands there yourself you might want to add some of us as collaborators, with appropriate permissions for what you would like us to help out with. Cheers, Kent.
On Fri, Jan 12, 2024 at 4:40 PM Kent Gibson <warthog618@gmail.com> wrote: > > On Fri, Jan 12, 2024 at 11:25:02PM +0800, Kent Gibson wrote: > > On Fri, Jan 12, 2024 at 04:05:46PM +0100, Bartosz Golaszewski wrote: > > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > The github page over at https://github.com/brgl/libgpiod has been reopened > > > for bug reports and discussions. Add a link and a mention to the README > > > file. > > > > > > > No problem with the patch, but the github page itself seems to have > > Issues and Discussions disabled. And the Wiki might be useful too? > > > > The Releases section could use updating - it lists the latest as v0.3 > from June 2017! > I don't want to do releases on github. The right place to fetch the releases from is kernel.org[1] (I should add this link to README too) and I don't want to duplicate release tarballs. I'm pretty sure people would start fetching releases from github for distros etc. Bart > And unless you intend to handle everything that lands there yourself you > might want to add some of us as collaborators, with appropriate permissions > for what you would like us to help out with. > > Cheers, > Kent. [1] https://mirrors.edge.kernel.org/pub/software/libs/libgpiod/
On Fri Jan 12, 2024 at 5:07 PM CET, Bartosz Golaszewski wrote: > On Fri, Jan 12, 2024 at 4:40 PM Kent Gibson <warthog618@gmail.com> wrote: > > > > On Fri, Jan 12, 2024 at 11:25:02PM +0800, Kent Gibson wrote: > > > On Fri, Jan 12, 2024 at 04:05:46PM +0100, Bartosz Golaszewski wrote: > > > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > > > The github page over at https://github.com/brgl/libgpiod has been reopened > > > > for bug reports and discussions. Add a link and a mention to the README > > > > file. > > > > > > > > > > No problem with the patch, but the github page itself seems to have > > > Issues and Discussions disabled. And the Wiki might be useful too? > > > > > > > The Releases section could use updating - it lists the latest as v0.3 > > from June 2017! > > > > I don't want to do releases on github. The right place to fetch the > releases from is kernel.org[1] (I should add this link to README too) > and I don't want to duplicate release tarballs. I'm pretty sure people > would start fetching releases from github for distros etc. You might want to delete the old tags from the github remote in that case. I think it might be a bit confusing to see v0.3 as "most recent" release otherwise. - Erik
On Fri, Jan 12, 2024 at 05:13:37PM +0100, Erik Schilling wrote: > On Fri Jan 12, 2024 at 5:05 PM CET, Bartosz Golaszewski wrote: > > On Fri, Jan 12, 2024 at 4:26 PM Erik Schilling > > <erik.schilling@linaro.org> wrote: > > > > > > disabled for the public. > > > > > > - Erik > > > > > > > Should be good now. > > Reviewed-by: Erik Schilling <erik.schilling@linaro.org> > > > Should I create a wiki too? > > Got no opinion on that. > I think the wiki could be useful for linking to things that aren't in the README or that you can't readily link to with a plain text README, like links to the readthedocs documentation, kernel patch submission process, maybe some hints on building for different distros (e.g. to identify packages that need to be installed first).... And speaking of the README, just noticed it mentions 4.8 as being the first release of the uAPI, but libgpiod uses v2 exclusively, and that wasn't introduced until 5.10. We might want to update that - still mention 4.8, since you also mention deprecating sysfs, but also mention v2 and 5.10. Cheers, Kent.
On Fri, Jan 12, 2024 at 05:07:42PM +0100, Bartosz Golaszewski wrote: > On Fri, Jan 12, 2024 at 4:40 PM Kent Gibson <warthog618@gmail.com> wrote: > > > > On Fri, Jan 12, 2024 at 11:25:02PM +0800, Kent Gibson wrote: > > > On Fri, Jan 12, 2024 at 04:05:46PM +0100, Bartosz Golaszewski wrote: > > > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > > > The github page over at https://github.com/brgl/libgpiod has been reopened > > > > for bug reports and discussions. Add a link and a mention to the README > > > > file. > > > > > > > > > > No problem with the patch, but the github page itself seems to have > > > Issues and Discussions disabled. And the Wiki might be useful too? > > > > > > > The Releases section could use updating - it lists the latest as v0.3 > > from June 2017! > > > > I don't want to do releases on github. The right place to fetch the > releases from is kernel.org[1] (I should add this link to README too) > and I don't want to duplicate release tarballs. I'm pretty sure people > would start fetching releases from github for distros etc. > Uugh, yeah github automatically provides tarballs for releases and there is no way to disable that. I was thinking you could just use it to advertise a particular tag as a release, but don't want the additional tarballs floating around. So, yeah, definitely need to add links to the official sources for tarballs, cos they can still download a zip of the tree, even if you haven't provided the tags. And, if you have time, adding an issue describing the daemon and where you are at would be really helpful. Though that is currently living in the topic/dbus branch of libgpiod-private, right? Cheers, Kent.
On Fri, Jan 12, 2024 at 5:18 PM Erik Schilling <erik.schilling@linaro.org> wrote: > > On Fri Jan 12, 2024 at 5:07 PM CET, Bartosz Golaszewski wrote: > > On Fri, Jan 12, 2024 at 4:40 PM Kent Gibson <warthog618@gmail.com> wrote: > > > > > > On Fri, Jan 12, 2024 at 11:25:02PM +0800, Kent Gibson wrote: > > > > On Fri, Jan 12, 2024 at 04:05:46PM +0100, Bartosz Golaszewski wrote: > > > > > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > > > > > > > > > The github page over at https://github.com/brgl/libgpiod has been reopened > > > > > for bug reports and discussions. Add a link and a mention to the README > > > > > file. > > > > > > > > > > > > > No problem with the patch, but the github page itself seems to have > > > > Issues and Discussions disabled. And the Wiki might be useful too? > > > > > > > > > > The Releases section could use updating - it lists the latest as v0.3 > > > from June 2017! > > > > > > > I don't want to do releases on github. The right place to fetch the > > releases from is kernel.org[1] (I should add this link to README too) > > and I don't want to duplicate release tarballs. I'm pretty sure people > > would start fetching releases from github for distros etc. > > You might want to delete the old tags from the github remote in that > case. I think it might be a bit confusing to see v0.3 as "most recent" > release otherwise. > Yeah, I did that and only left master, v2.1.x and v1.6.x branches alive. Bart
diff --git a/README b/README index 69128dc..41237f4 100644 --- a/README +++ b/README @@ -294,9 +294,14 @@ Those also provide examples of the expected formatting. Allow some time for your e-mail to propagate to the list before retrying, particularly if there are no e-mails in the list more recent than yours. +There is a libgpiod github page[7] available for reporting bugs and general +discussions and although PRs can be submitted and discussed, upstreambound +patches need to go through the mailing list nevertheless. + [1] https://github.com/kward/shunit2 [2] http://vger.kernel.org/vger-lists.html#linux-gpio [3] https://docs.kernel.org/process/email-clients.html [4] https://docs.kernel.org/process/coding-style.html [5] https://docs.kernel.org/process/submitting-patches.html [6] https://lore.kernel.org/linux-gpio/ +[7] https://github.com/brgl/libgpiod