On Tue, May 26, 2015 at 12:45 AM, G?bor Cs?rdi <csardi.gabor at gmail.com> wrote:> On Mon, May 25, 2015 at 8:28 PM, Simon Urbanek <simon.urbanek at r-project.org> > wrote: > >> One issue I have with this is that it doesn't point to the original GitHub >> repositories of the packages, so you end up with additional repositories on >> Github in Gabor's name that have nothing to do with the actual Github >> repositories of the packages. I understand that it's technically necessary, >> but I fear it will lead to a lot of confusion... > > > Well, we point to the original GitHub repo is that is given in the URL > field. It would be nice to have an "official" field for source code > repository in DESCRIPTION. > > But I agree with you that this has great potential for confusion. Several > people have been sending pull requests to github.com/cran repos, most of > them not realizing that they are not the right repos to fork. (Although > many packages are not on GH or any other similar service, and then are kind > of the places to fork.) > > I could have a large warning popup on the link from r-pkg.org, with red > flags, and you would see this before the actual repo. But this has its own > problems, like being annoying after a while, how to turn it off with > browser cookies, etc. > > The best would be to somehow have a warning on the GitHub repo pages, but > there isn't a lot I can modify there if I don't want to change/add the > README file, which would effectively change the package. I could probably > add 'WARNING: this is a read-only mirror, and not the original package > repository' to the one-line description on the top. > > If you have other ideas, please let me know.If people send pull requests, maybe adding a generic open pull request to each repository with title "MIRROR ONLY: Do not send pull requests here" would help. The fancy version would be to say "MIRROR ONLY: All patches/pull requests should be sent to <URL>", where <URL> is from the DESCRIPTION field 'URL'. That might prevent a few more. <blink>You can boost it with lots of colorful "WARNING", "NO TRESPASSING", ... labels as well</blink>. The next level up is to have a service that automatically reject pull requests with an informative error message. /Henrik> > Gabor > > >> On May 24, 2015, at 5:44 AM, Rainer M Krug <Rainer at krugs.de> wrote: >> >> > G?bor Cs?rdi <csardi.gabor at gmail.com> writes: >> > >> >> Dear All, >> >> >> >> [ I was wondering if this should have gone to the new mailing list. >> Maybe. ] >> >> >> >> As some of you maybe know from my earlier posts, I am building a simple >> >> search engine for R packages. Now the search engine has a proper web >> site, >> >> where you can also browse CRAN packages. >> >> >> >> http://www.r-pkg.org/ >> >> >> >> As I see the value is in >> >> 1. package search (search box on top right) >> >> 2. APIs, see http://www.r-pkg.org/services >> >> >> >> It is in alpha version, meaning that things seem to work, some pages >> are a >> >> bit slow and there are a lot of glitches to fix. >> > >> > I had a quick peek, and it looks really nice! I particularly think the >> > github integration for diff-ing versions can be very use full! >> > >> > It might be an idea, to also add R itself to the github repo for >> > diff-ing? >> > >> > Thanks a lot, >> > >> > Rainer >> > >> >> >> >> Please tell me what you think. >> >> >> >> Best, >> >> Gabor >> >> >> >> [[alternative HTML version deleted]] >> >> >> > >> > -- >> > Rainer M. Krug >> > email: Rainer<at>krugs<dot>de >> > PGP: 0x0F52F982 >> > ______________________________________________ >> > R-devel at r-project.org mailing list >> > https://stat.ethz.ch/mailman/listinfo/r-devel >> >> > > [[alternative HTML version deleted]] > > ______________________________________________ > R-devel at r-project.org mailing list > https://stat.ethz.ch/mailman/listinfo/r-devel
On Tue, May 26, 2015 at 1:43 PM, Henrik Bengtsson <henrik.bengtsson at ucsf.edu> wrote:[...]> If people send pull requests, maybe adding a generic open pull request > to each repository with title "MIRROR ONLY: Do not send pull requests > here" would help. The fancy version would be to say "MIRROR ONLY: All > patches/pull requests should be sent to <URL>", where <URL> is from > the DESCRIPTION field 'URL'. That might prevent a few more. >The problem with that 1) if I put in ~7000 fake pull requests, then it is messy for me to find the "real" pull requests, put in by other people, and 2) I am pretty sure that many people do not look at previous pull requests, when they put in theirs. But it is so far not such a big deal, I only got ~50 of them during ~12 months. And in many cases, they are useful, because they can be moved upstream. I will write a short tutorial about that. Gabor [[alternative HTML version deleted]]
Or maybe it would be sensible to ask GitHub if they can fix this. If it's a common-ish use case (e.g. for mirrors), it's not something that should be terribly challenging engineering-wise, and it would prevent a lot of hooha. Funfact: GitHub is run and staffed by actual people, most of them pretty cool Statistics is the grammar of science. Karl Pearson <http://en.wikipedia.org/wiki/The_Grammar_of_Science> On Tue, May 26, 2015 at 10:43 AM, Henrik Bengtsson < henrik.bengtsson at ucsf.edu> wrote:> On Tue, May 26, 2015 at 12:45 AM, G?bor Cs?rdi <csardi.gabor at gmail.com> > wrote: > > On Mon, May 25, 2015 at 8:28 PM, Simon Urbanek < > simon.urbanek at r-project.org> > > wrote: > > > >> One issue I have with this is that it doesn't point to the original > GitHub > >> repositories of the packages, so you end up with additional > repositories on > >> Github in Gabor's name that have nothing to do with the actual Github > >> repositories of the packages. I understand that it's technically > necessary, > >> but I fear it will lead to a lot of confusion... > > > > > > Well, we point to the original GitHub repo is that is given in the URL > > field. It would be nice to have an "official" field for source code > > repository in DESCRIPTION. > > > > But I agree with you that this has great potential for confusion. Several > > people have been sending pull requests to github.com/cran repos, most of > > them not realizing that they are not the right repos to fork. (Although > > many packages are not on GH or any other similar service, and then are > kind > > of the places to fork.) > > > > I could have a large warning popup on the link from r-pkg.org, with red > > flags, and you would see this before the actual repo. But this has its > own > > problems, like being annoying after a while, how to turn it off with > > browser cookies, etc. > > > > The best would be to somehow have a warning on the GitHub repo pages, but > > there isn't a lot I can modify there if I don't want to change/add the > > README file, which would effectively change the package. I could probably > > add 'WARNING: this is a read-only mirror, and not the original package > > repository' to the one-line description on the top. > > > > If you have other ideas, please let me know. > > If people send pull requests, maybe adding a generic open pull request > to each repository with title "MIRROR ONLY: Do not send pull requests > here" would help. The fancy version would be to say "MIRROR ONLY: All > patches/pull requests should be sent to <URL>", where <URL> is from > the DESCRIPTION field 'URL'. That might prevent a few more. > <blink>You can boost it with lots of colorful "WARNING", "NO > TRESPASSING", ... labels as well</blink>. The next level up is to > have a service that automatically reject pull requests with an > informative error message. > > /Henrik > > > > > > Gabor > > > > > >> On May 24, 2015, at 5:44 AM, Rainer M Krug <Rainer at krugs.de> wrote: > >> > >> > G?bor Cs?rdi <csardi.gabor at gmail.com> writes: > >> > > >> >> Dear All, > >> >> > >> >> [ I was wondering if this should have gone to the new mailing list. > >> Maybe. ] > >> >> > >> >> As some of you maybe know from my earlier posts, I am building a > simple > >> >> search engine for R packages. Now the search engine has a proper web > >> site, > >> >> where you can also browse CRAN packages. > >> >> > >> >> http://www.r-pkg.org/ > >> >> > >> >> As I see the value is in > >> >> 1. package search (search box on top right) > >> >> 2. APIs, see http://www.r-pkg.org/services > >> >> > >> >> It is in alpha version, meaning that things seem to work, some pages > >> are a > >> >> bit slow and there are a lot of glitches to fix. > >> > > >> > I had a quick peek, and it looks really nice! I particularly think the > >> > github integration for diff-ing versions can be very use full! > >> > > >> > It might be an idea, to also add R itself to the github repo for > >> > diff-ing? > >> > > >> > Thanks a lot, > >> > > >> > Rainer > >> > > >> >> > >> >> Please tell me what you think. > >> >> > >> >> Best, > >> >> Gabor > >> >> > >> >> [[alternative HTML version deleted]] > >> >> > >> > > >> > -- > >> > Rainer M. Krug > >> > email: Rainer<at>krugs<dot>de > >> > PGP: 0x0F52F982 > >> > ______________________________________________ > >> > R-devel at r-project.org mailing list > >> > https://stat.ethz.ch/mailman/listinfo/r-devel > >> > >> > > > > [[alternative HTML version deleted]] > > > > ______________________________________________ > > R-devel at r-project.org mailing list > > https://stat.ethz.ch/mailman/listinfo/r-devel > > ______________________________________________ > R-devel at r-project.org mailing list > https://stat.ethz.ch/mailman/listinfo/r-devel >[[alternative HTML version deleted]]