From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Original-To: caml-list@sympa.inria.fr Delivered-To: caml-list@sympa.inria.fr Received: from mail2-relais-roc.national.inria.fr (mail2-relais-roc.national.inria.fr [192.134.164.83]) by sympa.inria.fr (Postfix) with ESMTPS id 566737F890 for ; Tue, 25 Mar 2014 17:40:07 +0100 (CET) Received-SPF: None (mail2-smtp-roc.national.inria.fr: no sender authenticity information available from domain of anil@recoil.org) identity=pra; client-ip=89.16.177.154; receiver=mail2-smtp-roc.national.inria.fr; envelope-from="anil@recoil.org"; x-sender="anil@recoil.org"; x-conformance=sidf_compatible Received-SPF: None (mail2-smtp-roc.national.inria.fr: no sender authenticity information available from domain of anil@recoil.org) identity=mailfrom; client-ip=89.16.177.154; receiver=mail2-smtp-roc.national.inria.fr; envelope-from="anil@recoil.org"; x-sender="anil@recoil.org"; x-conformance=sidf_compatible Received-SPF: None (mail2-smtp-roc.national.inria.fr: no sender authenticity information available from domain of postmaster@dark.recoil.org) identity=helo; client-ip=89.16.177.154; receiver=mail2-smtp-roc.national.inria.fr; envelope-from="anil@recoil.org"; x-sender="postmaster@dark.recoil.org"; x-conformance=sidf_compatible X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: AuQDAK6wMVNZELGagGdsb2JhbABZg0GrHpBxhzWBMQ4BAQsUBz6CJQEBAQMBAQEBNw4mCwULCy0ZIQYBLwYTFIYJgUgDCQwJsUSWSA2HMheMUhyBHhEBHTMHCoMagRQElGGBFWqBbYEyizaIeD2BLAkXgRk X-IPAS-Result: AuQDAK6wMVNZELGagGdsb2JhbABZg0GrHpBxhzWBMQ4BAQsUBz6CJQEBAQMBAQEBNw4mCwULCy0ZIQYBLwYTFIYJgUgDCQwJsUSWSA2HMheMUhyBHhEBHTMHCoMagRQElGGBFWqBbYEyizaIeD2BLAkXgRk X-IronPort-AV: E=Sophos;i="4.97,728,1389740400"; d="scan'208";a="64690747" Received: from recoil.dh.bytemark.co.uk (HELO dark.recoil.org) ([89.16.177.154]) by mail2-smtp-roc.national.inria.fr with SMTP; 25 Mar 2014 17:40:06 +0100 Received: (qmail 18533 invoked by uid 634); 25 Mar 2014 16:40:05 -0000 X-Spam-Level: * X-Spam-Check-By: dark.recoil.org Received: from cpc7-cmbg14-2-0-cust238.5-4.cable.virginm.net (HELO [192.168.1.110]) (86.30.244.239) (smtp-auth username remote@recoil.org, mechanism cram-md5) by dark.recoil.org (qpsmtpd/0.84) with ESMTPA; Tue, 25 Mar 2014 16:40:05 +0000 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 7.2 \(1874\)) From: Anil Madhavapeddy In-Reply-To: Date: Tue, 25 Mar 2014 16:40:01 +0000 Cc: caml users , "platform@lists.ocaml.org" Content-Transfer-Encoding: quoted-printable Message-Id: References: To: Gabriel Scherer X-Mailer: Apple Mail (2.1874) X-Virus-Checked: Checked by ClamAV on dark.recoil.org Subject: Re: [Caml-list] [ANN] Experiment: OCaml patch review on github.com/ocaml/ocaml As a followup to this, I've written a script that syncs all the open pull requests against the compiler as OPAM compiler switches. This is now running live on the central OPAM respository. For instance, if you now `opam update` and look at your switches: $ opam switch --all -- -- 4.02.0dev+pr10 Add String.{split,rsplit} -- -- 4.02.0dev+pr13 Add String.{cut,rcut}. -- -- 4.02.0dev+pr14 Add absolute directory names to bytec= ode format for ocamldebug to use -- -- 4.02.0dev+pr15 replace String.blit by String.unsafe_= blit -- -- 4.02.0dev+pr17 Cmm arithmetic optimisations -- -- 4.02.0dev+pr18 Patch for issue 5584 -- -- 4.02.0dev+pr2 Parse -.x**2. (unary -.) as -.(x**2.)= . Fix PR#3414 -- -- 4.02.0dev+pr20 OCamlbuild: Fix the check of ocamlfind -- -- 4.02.0dev+pr3 Extend record punning to allow destru= cturing. -- -- 4.02.0dev+pr4 Fix for PR#4832 (Filling bigarrays ma= y block out runtime) -- -- 4.02.0dev+pr6 Warn user when a type variable in a t= ype constraint has been instantiated. -- -- 4.02.0dev+pr7 Extend ocamllex with actions before r= efilling -- -- 4.02.0dev+pr8 Adds a .gitignore to ignore all gener= ated files during `make world.opt' -- -- 4.02.0dev+pr9 FreeBSD 10 uses clang by default, wit= h gcc not available by default -- -- 4.02.0dev+trunk latest trunk snapshot Each switch corresponds to the current development trunk, with the diff in the PR applied. If the patch is sane, you can proceed to install OPAM packages in the experimental tree as usual without affecting your day-to-day compiler switch. Hope this is useful! More details at: http://anil.recoil.org/2014/03/25/ocaml-github-and-opam.html It's set to run daily at the moment, and switches will be deleted once the corresponding pull request is closed. cheers, Anil On 30 Jan 2014, at 11:34, Gabriel Scherer wrote: > TL;DR: During the six next months, we will follow pull requests (PR) > posted on the github mirror of the OCaml distribution, as an > alternative to the mantis bugtracker. This experiment hopes to attract > more people to participate in the extremely helpful and surprisingly > rewarding activity of patch reviews. >=20 >=20 > Dear OCaml community, >=20 > I think we need more people ready to review patches proposed for > inclusion in the OCaml compiler/distribution; lack of reviews is > currently one of the bottleneck in the development process -- among > others, such as the sheer difficulty to reach consensus on any change > to the language itself. Doing patch reviews is helpful, extremely > interesting, and an excellent way to get to know more about small > parts of the compiler. >=20 > There was a resurgence of discussions on caml-list (Yotam Barnoy's > [moving to github] and Adrien Nader's thoughtful proposal of > a [mailing-list for patch review]). Amir Chaudhry launched a poll to > record decreasing order of preference, and the [results] are > clear-cut: people hate Mantis' guts, and would rather use anything > else. >=20 > [moving to github]: http://alan.petitepomme.net/cwn/2013.12.24.html#5 > [mailing-list for patch review]: > https://sympa.inria.fr/sympa/arc/caml-list/2014-01/msg00055.html > [results]: https://docs.google.com/forms/d/1QWhqJRv1yPvdi6E3AiqbvUwlqGorV= _Wbk7h_JYuDUiQ/viewanalytics >=20 > I declare open the following experiment: for six months, starting > today upto late July, patches proposed for the OCaml distribution may > be submitted as a pull request (PR) on the [main github mirror], and > we warmly encourage anyone to review the proposed patches, and make > any comments they feel can help. Anything that can help improve the > contribution, or discuss potential issues (backward compatibility, > future-proofiness of the change, alternative designs...) will speed up > the time between a patch proposal and a clear decision to integrate it > or not. >=20 > [main github mirror]: https://github.com/ocaml/ocaml/ >=20 > In six months, we will reconsider, the default choice being to stop > using github and revert to a mantis-only workflow. In the meantime, > I will mirror the github PRs on the mantis side, so that contributors > that do not wish to use the github interface can continue working as > before. Patches and reviews are of course still welcome on mantis. >=20 > Note that github will *not* be used for issue tracking, only for patch > reviews. If you want to submit a patch against a bug discussed in > Mantis, or want to re-submit a patch already in Mantis (in the wild > hope of more eyeballs), feel free to send a github PR and link to it > from the bugtracker. Finally, the github mirror remains *read-only*: > if patches are accepted, the PR will be closed but will be committed > to the SVN first, and synced in git as usual. >=20 > We're just trying things to see if it works better. I hope it does. In > any case, thanks in advance for your participation -- whichever tool > you use. Happy hacking! >=20 > . >=20 > PS: If you want to get notified for all Pull Requests sent, you > (need a github account and) can click on the "Watch" button in the top > right of http://github.com/ocaml/ocaml to register for > notifications. In the [notification settings] page of your account, > you can set up notifications to get send by email and/or to the > (mostly useless) github notification web interface. >=20 > [notification settings](https://github.com/settings/notifications) > _______________________________________________ > Platform mailing list > Platform@lists.ocaml.org > http://lists.ocaml.org/listinfo/platform >=20