<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 22 Jan 2020 at 01:04, Xavier <<a href="mailto:yadd@debian.org">yadd@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Le 21/01/2020 à 20:23, Xavier a écrit :<br>
> Le 21/01/2020 à 19:18, Nilesh Patra a écrit :<br>
>><br>
>><br>
>> On Tue, 21 Jan 2020 at 23:11, Xavier <<a href="mailto:x.guimard@free.fr" target="_blank">x.guimard@free.fr</a><br>
>> <mailto:<a href="mailto:x.guimard@free.fr" target="_blank">x.guimard@free.fr</a>>> wrote:<br>
>><br>
>>     Le 21/01/2020 à 16:08, Nilesh Patra a écrit :<br>
>>     ><br>
>>     >     Hi,<br>
>>     ><br>
>>     >     please regenerate minified files (exclude + repack): launch<br>
>>     make in<br>
>>     >     subdirectories<br>
>>     ><br>
>>     ><br>
>>     > Did the changes as asked. Could you review now?<br>
>>     > Thanks!<br>
>><br>
>>     You miss to exclude also lib/d3/*.min.js<br>
>><br>
>><br>
>> Right. However, lib's Makefile uses certain files from src/ directory<br>
>> that aren't present, and there is no rule to do that, either in any of<br>
>> the Makefiles. (For example: src/core/date.js, src/core/zip.js etc)<br>
>> What would be the best to do in this case? I was thinking of keeping the<br>
>> minified files, as is and dropping the Makefile(inside lib/d3/), file an<br>
>> issue upstream.<br>
>> Do you think it is the right thing to do, or should something else be<br>
>> done instead? (Need your guidance here)<br>
> <br>
> Then keep lib/d3*.js without lib/d3*.min.js<br>
<br></blockquote><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
And generate them using uglifyjs<br></blockquote><div>Nice! :)<br></div><div><br></div><div>Done, and pushed to my local fork. Please Review, if it looks OK.</div><div>Thanks!</div><div>Nilesh  </div></div></div>