On 23/02/15 15:16, Daniel Kolesa wrote: > On Wed, Feb 18, 2015 at 9:43 PM, perepelits.m <perepelit...@samsung.com> > wrote: > >> cedric pushed a commit to branch master. >> >> >> http://git.enlightenment.org/core/efl.git/commit/?id=2e2145b5945c94587e5a11502432ea2e8f75d2bd >> >> commit 2e2145b5945c94587e5a11502432ea2e8f75d2bd >> Author: perepelits.m <perepelit...@samsung.com> >> Date: Wed Feb 18 22:00:58 2015 +0100 >> >> evas: Evas_3D - improve Shooter example. >> >> Summary: This example includes some previous developments to create >> 3d-world mechanics. >> >> Reviewers: cedric, raster, Hermet >> >> Subscribers: cedric, artem.popov >> >> Differential Revision: https://phab.enlightenment.org/D1991 >> > > Dude, what the hell? One does *NOT* simply commit 50MB of worthless (and > mostly binary) assets into a Git repository (the space in the directory > name is the least bad thing about this), *ESPECIALLY* our primary EFL > repository; this is really bad (especially as the example looks like shit). > I would say this deserves a revert, but sadly, it's forever written in our > history and now *everyone* will be pulling the extra fucking 50MB... i hope > you're happy. > > Do *NOT* do this kind of shit again.
Yeah, Cedric, this is crazy! We've talked about it before with relation to exactness! Pushing this amount of crazy useless data just for the sake of better examples is crazy and stupid. As Daniel said, the space in the directory name and the use of capital letters, while major offences on their own, are nothing in comparison to this shit. Have you even reviewed this crazy patch before pushing it?! It's still worth reverting this, as at least people doing shallow clones won't have to suffer this craziness. Cedric, please revert this and never do this shit again. :( Make a 3d examples repo if you are so keen to have one. **Please revert this** Just to repeat this again: while nothing in comparison to the massive binary dump, the space in the filename and the use of caps is enough to reject this. What's the point of even reviewing patches if you don't stop obvious craziness at the door? -- Tom. ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
