1.2.1 patch release with this bug fix (+ others if applicable) will be very useful for the users.
On Tue, May 22, 2018 at 1:08 PM, Rahul Huilgol <rahulhuil...@gmail.com> wrote: > Maybe we could do that now, after the code for the release has been voted > on. We could maintain a patches branch for a release. This would also be > helpful for users who are using a particular release version but hesitate > to switch to the latest release when it's out because of the many changes. > Such users I've seen maintain their own patches branch. We could help them > and keep things consistent. We can cut a patch release from that if we have > many fixes or important fixes. > > On Thu, May 17, 2018 at 10:05 PM, Anirudh <anirudh2...@gmail.com> wrote: > > > Thats an interesting suggestion. I understand the benefits, but this will > > complicate things if we want to do another quick release like the > previous > > one (for eg. if there is a license issue because of which the release is > > cancelled). > > > > Anirudh > > > > On Thu, May 17, 2018 at 9:23 PM, Naveen Swamy <mnnav...@gmail.com> > wrote: > > > > > why don't we cherry-pick and push it to the branch as well ? next time > a > > > release is cut from that branch we'll have everything working. > > > > > > On Thu, May 17, 2018 at 9:21 PM, Anirudh <anirudh2...@gmail.com> > wrote: > > > > > > > Correction : This fix went into master but not 1.2 release. > > > > > > > > On Thu, May 17, 2018 at 9:21 PM, Anirudh <anirudh2...@gmail.com> > > wrote: > > > > > > > > > Hi Xinyu, > > > > > > > > > > Thank you! This fix didnt went into master but not 1.2 release. > > > > > > > > > > Anirudh > > > > > > > > > > On Thu, May 17, 2018 at 5:23 PM, Chen, Xinyu1 < > xinyu1.c...@intel.com > > > > > > > > wrote: > > > > > > > > > >> Hi all, > > > > >> > > > > >> Regarding to the following knowing issues listed below RC3: > > > > >> > > > > >> > > > > >> * CMake build ignores the USE_MKLDNN flag and doesn't build > with > > > > >> MKLDNN support even with -DUSE_MKLDNN=1. To workaround the issue > > > please > > > > >> see: #10801<https://github.com/apache/incubator-mxnet/issues/ > 10801 > > >. > > > > >> I think this problem has already been fixed in #10629< > > > > >> https://github.com/apache/incubator-mxnet/pull/10629> and we can > > use > > > > >> CMake to build MXNet with MKLDNN support on WIN32 platform now. > > > > >> Xinyu > > > > >> > > > > >> > > > > > > > > > > > > > > > > > > -- > Rahul Huilgol > -- Sandeep Krishnamurthy