Hi Willem, Would you mind proposing a pull request with that change ? It looks good to me. Cheers On 30/03/2016 11:25, Willem Jan Withagen wrote: > On 29-3-2016 17:12, Willem Jan Withagen wrote: >> On 29-3-2016 14:28, Loic Dachary wrote: >>> Hi Willem, >>> >>> This has undergone various fixes all of them wrong in a context or >>> another. The last of them is (hopefully ;-) >>> https://github.com/ceph/ceph/pull/8320 >> >> Hi Loic, >> >> I watched the discussion carefully, and was under the assemption that >> everything was already merged. But I'll take your pull and see what happens. > > Hi Loic, > > I saw that #8320 was merged and rebased. So running/compiling to see > what brings. > > There might be one change that you'd care to take along in this as well, > which I have to keep my tests running ATM. > Otherwise I'll just keep it in my patchset to the time to come. > > --WjW > > : ${CEPH_DETECT_INIT_VIRTUALENV:=ceph-detect-init-virtualenv} > > case `uname` in > FreeBSD) > echo FreeBSD init system has not been integrated. > exit 0 > ;; > *) > # run from the ceph-detect-init directory or from its parent > test -d ceph-detect-init && cd ceph-detect-init > source ${CEPH_DETECT_INIT_VIRTUALENV}/bin/activate > tox > ${CEPH_DETECT_INIT_VIRTUALENV}/tox.out 2>&1 > status=$? > grep -v InterpreterNotFound < ${CEPH_DETECT_INIT_VIRTUALENV}/tox.out > exit $status > ;; > esac > > -- Loïc Dachary, Artisan Logiciel Libre -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
