[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#912444: stretch-pu: package ola/0.10.3.nojsmin-2



On Thu, Nov 01, 2018 at 08:28:29PM +0000, Adam D. Barratt wrote:
> Control: tags -1 + confirmed
> 
> On Wed, 2018-10-31 at 17:19 +0100, Wouter Verhelst wrote:
> > Would like to fix #876251; minor but very annoying bug in init script
> > means the daemon just doesn't start.
> 
> Please go ahead.

Thanks.

However, after discussing this a bit more with upstream, I found there's
actually a second error in the package; upstream ships pre-minified
javascript files, which we remove, but in one case the link to the
correct file was not updated in the html that we ship, resulting in no
functionality for that particular bit (also the RDM test server).

To fix that, on top of the debdiff that I sent before, the following
would be necessary:

diff --git a/tools/rdm/static/rdmtests.html b/tools/rdm/static/rdmtests.html
index c2ba01fde..47caea99e 100644
--- a/tools/rdm/static/rdmtests.html
+++ b/tools/rdm/static/rdmtests.html
@@ -4,7 +4,7 @@
     <link type='text/css' rel='stylesheet' href='/static/jquery-ui-1.8.21.custom.css' />
     <link rel='stylesheet' href='/static/ui.multiselect.css' type='text/css'>
     <script src='/static/jquery.min.js'></script>
-    <script src='/static/jquery-ui.custom.min.js'></script>
+    <script src='/static/jquery-ui.min.js'></script>
     <script src='/static/ui.multiselect.js'></script>
     <script src='/static/rdm_tests.js'></script>
     <title>RDM Tests &amp; Publisher</title>

(plus obviously a changelog entry)

OK?

-- 
To the thief who stole my anti-depressants: I hope you're happy

  -- seen somewhere on the Internet on a photo of a billboard


Reply to: