<div dir="ltr">Oh my, I didn't expect that.<div><br></div><div>Turns out "dh_ruby test" doesn't do anything... Oh my. </div><div>I updated the patch to compensate. It works now for both the full build (in pbuilder) and with nocheck (and qunit-selenium not getting installed).</div><div><br></div><div>Cheers,</div><div>Sven</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Mi., 2. Dez. 2020 um 00:09 Uhr schrieb Antonio Terceiro <<a href="mailto:terceiro@debian.org">terceiro@debian.org</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Nov 09, 2020 at 03:40:15PM +0100, Sven Mueller wrote:<br>
> Package: rails<br>
> Version: 2:6.0.3.4+dfsg-1<br>
> Tags: patch<br>
> <br>
> Turns out that this isn't needed and pulls in a lot of dependencies if the<br>
> build is done without tests.<br>
> <br>
> The attached patch moves the start of the redis server (only needed for<br>
> tests) inside the override_dh_auto_test section in debian/rules (I suspect<br>
> that the redis server was also only pulled in via the qunit-selenium<br>
> dependency? - without this move, the resulting build failed). The<br>
> dependency on qunit-selenium is marked as <!nocheck> so it is now only<br>
> pulled in if tests are enabled.<br>
> <br>
> Please consider merging this. The change is trivial and I don't believe it<br>
> would create any copyright concerns, but just in case a license would<br>
> actually be required, I license this under the Expat license or CC-BY 3.0<br>
> or CC-BY-SA-4.0, the choice of license is left to the receiver of the<br>
> license and should  be fully compatible with the Debian packaging as well<br>
> as upstream (though this only affects the packaging, so I doubt they care).<br>
<br>
> diff --git a/debian/control b/debian/control<br>
> index 59d14bfe..3d067e13 100644<br>
> --- a/debian/control<br>
> +++ b/debian/control<br>
> @@ -18,7 +18,7 @@ Build-Depends: debhelper-compat (= 12),<br>
>                 ruby-byebug,<br>
>  # Ruby packages - from Gemfile<br>
>                 puma (>= 4.1~),<br>
> -               qunit-selenium,<br>
> +               qunit-selenium <!nocheck>,<br>
>                 racc (>= 1.4.6),<br>
>                 rake (>= 11.1),<br>
>                 ruby-bcrypt (<< 3.2),<br>
> diff --git a/debian/rules b/debian/rules<br>
> index d3747fee..41bfe45a 100755<br>
> --- a/debian/rules<br>
> +++ b/debian/rules<br>
> @@ -20,17 +20,17 @@ override_dh_clean:<br>
>       debian/stop-redis-server.sh || true<br>
>       <br>
>  override_dh_auto_install:<br>
> -     # start redis server for tests<br>
> -     debian/start-redis-server.sh<br>
>       # auto install<br>
>       dh_auto_install -O--buildsystem=ruby<br>
> -     # kill redis server used for tests<br>
> -     debian/stop-redis-server.sh     <br>
>       $(RM) debian/ruby-activesupport/usr/bin/generate_tables<br>
>       $(RM) debian/*/usr/bin/test<br>
>       rmdir debian/*/usr/bin || true<br>
>  <br>
>  override_dh_auto_test:<br>
>  ifeq ($(filter nocheck,$(DEB_BUILD_PROFILES)),)<br>
> +     # start redis server for tests<br>
> +     debian/start-redis-server.sh<br>
>       dh_auto_test<br>
> +     # kill redis server used for tests<br>
> +     debian/stop-redis-server.sh     <br>
>  endif<br>
<br>
Did you test this on a full build? gem2deb has a limitation/bug that<br>
causes the tests to be executed during the install target, and not the<br>
test one.<br>
</blockquote></div>