<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 15, 2013 at 7:04 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hello list<br>
<br>
Feeling inspired by the automake work done in mesa, I felt like<br>
finishing a few things that did not receive too much attention<br>
 * use Makefile.sources wherever possible<br>
 * cleanup the duplicated C{,PP,XX}FLAGS and factor out the the common<br>
ones into Automake.inc<br>
<br>
If anyone is interested I have pushed my preliminary work to the<br>
makefile.sources branch at <a href="https://github.com/evelikov/Mesa/" target="_blank">https://github.com/evelikov/Mesa/</a><br>
<br>
Currently I have gone through the following:<br>
 gallium/drivers<br>
 gallium/state_trackers/dri/drm<br>
 gallium/state_trackers/dri/sw<br>
 gallium/state_trackers/vega<br>
 gallium/state_trackers/xorg<br>
<br>
Producing the following diff stat:<br>
 54 files changed, 352 insertions(+), 552 deletions(-)<br>
<br>
<br>
Pros:<br>
* patches such as "gallium/dri-targets: hide all symbols except for<br>
__driDriverExtensions" will be brought down to changing only with 2-3 lines<br>
* one less chance of breaking builds when someone forgets to update the<br>
SCons/Makefile.am/etc build, after adding/removing a file<br>
<br>
Cons:<br>
* Non nouveau changes will be only(lacking any other the hardware atm).<br>
Note that I will check the symbols for all drivers, to ensure that I do<br>
not cause chaos.<br>
<br>
Curious if anyone is interested and have any comments on this.<br>
Note: there may be some git rebase fails as I've started this ~3months ago<br></blockquote><div><br></div><div>This seems to be have dropped on the floor.</div><div><br></div><div>The Makefile.sources patches looks good, can't comment on the other ones.</div>
<div><br></div><div>If you rebase them and you clobber the Makefile.sources ones, then send that</div><div>along with the reset of the patches on that branch out I'm sure they will get reviewed.</div><div><br></div><div>
Cheers, Jakob.</div><div><br></div><div><br></div></div></div></div>