On Mon, May 11, 2015 at 10:55 AM, John Ralls <[email protected]> wrote:
> > > On May 11, 2015, at 10:21 AM, Philip Chimento <[email protected]> > wrote: > > > > On Mon, May 11, 2015 at 7:18 AM, John Ralls <[email protected]> wrote: > > > >> On May 11, 2015, at 1:50 AM, Kirill Moizik <[email protected]> wrote: > >> > >> Hi John, > >> I sent it here https://github.com/GNOME/gtk-osx/pulls > >> I am a little bit confused about all this different projects > >> Kirill > >> > >> On Sun, May 10, 2015 at 6:59 PM, John Ralls <[email protected]> wrote: > >> > >>> On May 10, 2015, at 1:43 AM, Kirill Moizik <[email protected]> wrote: > >>> > >>> Thanks for your reply John, > >>> i sent pull request with the fix, > >>> please review it > >>> Thanks in advance, > >> > >> Please remember to copy the list on all replies, using “reply all”. > >> > >> I don’t see any PRs on https://github.com/jralls/gtk-osx-build. Where > did you send it? > > > > Please remember to CC the list on all replies. > > > > Huh, I’d forgotten about that repo. Not a good choice for PRs because I > don’t get notifications about them, either by mail or on my dashboard. > Better to use my repo. > > > > The structure is a bit weird because of the history of gtk-osx. When > Richard Hult started it he created it separately from Gnome, and for a long > time the Gnome folks were passively to aggressively hostile about building > on anything other than Linux. That changed a few years ago when some of > them discovered Macs, and I was invited to move the projects to Gnome and > given push privs there. But git.gnome.org doesn’t have any way to handle > PRs, so I kept my Github repos open and push everything to both Gnome and > Github. > > > > The Gnome Github mirrors are just pushes from git.gnome.org, run by a > commit hook. There’s a FAQ about them at > https://wiki.gnome.org/Sysadmin/GitHub, the first of which says that PRs > aren’t accepted. We’ll see if anyone notices yours and my comment on it. > > > > There's a thread from just a few days ago on desktop-devel-list [1] > about changing to a system where it's the maintainer's choice whether to > accept pull requests or not; so I guess it's up to you, John. > > > > If you want to get notified about pull requests in the Gnome mirror as > well, you can "watch" the repo with the button at the upper right. > > > > [1] > https://mail.gnome.org/archives/desktop-devel-list/2015-May/msg00004.html > > Wow, I read that thread completely differently. Colin likes PRs and > Bastien and Emmanuelle hate them, though Emmanuelle grudgingly accepts that > they’re better for big change sets like yours yesterday. I didn’t see > anything that suggested TPTB were going to make it possible to handle PRs. > Whoops, I see where I was misleading - by "accept PRs" I didn't mean "click the Github merge button and have them committed into the GNOME codebase." What I read out of that thread was that they were going to make it possible for each repo's maintainer to individually opt in or out of saying "I will acknowledge and review Github PRs" or "PRs will be ignored, use Bugzilla." AFAIK you still have to do the actual commit to git.gnome.org, not to Github, but that's not too painful as Emmanuele pointed out. I guess that's what Colin does with ostree. Regards, -- Philip
_______________________________________________ Gtk-osx-devel-list mailing list [email protected] https://mail.gnome.org/mailman/listinfo/gtk-osx-devel-list
