<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 2, 2016 at 4:22 AM, Kohei Yoshida <span dir="ltr"><<a href="mailto:libreoffice@kohei.us" target="_blank">libreoffice@kohei.us</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Josh,<br>
<span class=""><br>
On Sat, 2016-01-30 at 07:21 -0500, joshbaldwin42 . wrote:<br>
<br>
> I am working on<br>
> <a href="https://bugs.documentfoundation.org/show_bug.cgi?id=66754" rel="noreferrer" target="_blank">https://bugs.documentfoundation.org/show_bug.cgi?id=66754</a> .  This<br>
> required a change to /setup_native/scripts/<a href="http://admin.pl" rel="noreferrer" target="_blank">admin.pl</a>.  I did<br>
><br>
> egrep -ir "<a href="http://admin.pl" rel="noreferrer" target="_blank">admin.pl</a>" .<br>
<br>
</span>BTW, when grepping through git-managed files, it's normally a lot faster<br>
to run 'git grep' than the normal grep command (in case you were not<br>
aware of it).<br>
<span class="">><br>
> to see where in libreoffice <a href="http://admin.pl" rel="noreferrer" target="_blank">admin.pl</a> used.  However, it does not<br>
> appear to be called anywhere.<br>
<br>
</span>Yup, it looks that way.  Running git log -S'<a href="http://admin.pl" rel="noreferrer" target="_blank">admin.pl</a>' reveals that this<br>
commit<br>
<br>
<a href="http://cgit.freedesktop.org/libreoffice/core/commit/?id=bdd97ec6ee5ad96fd2452ce99b3ebd28e540629a" rel="noreferrer" target="_blank">http://cgit.freedesktop.org/libreoffice/core/commit/?id=bdd97ec6ee5ad96fd2452ce99b3ebd28e540629a</a><br>
<br>
removed the last use of this script.  I'm not an expert on the packaging<br>
part of the build system, but I'm pretty certain that this script is<br>
unused for real.<br>
<br>
But just in case it is still used externally, let me CC Andras who is<br>
more familiar with this part of the code.<br>
<span class=""><br>
> I wanted to know so I can properly test the code before submitting.<br>
<br>
</span>I think you've done your due diligence to make sure the script is now<br>
unused.  Unless someone else screams and tells you "Please don't remove<br>
it!", I would just remove that file in your gerrit patch, to at least<br>
have Jenkins tell us if it still is buildable on all major platforms.<br>
<br></blockquote><div><br></div><div>I've never used <a href="http://admin.pl">admin.pl</a>. I think its functionality is 100% covered by 'msiexec -a' command, so we don't need it.<br><br></div><div>Best regards,<br></div><div>Andras <br></div></div></div></div>