Debian Bug report logs - #817126
grafana: FTBFS when built with dpkg-buildpackage -A (unable to chdir to _build)

Reply or subscribe to this bug.

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Santiago Vila <sanvila@debian.org>

To: Debian BTS <submit@bugs.debian.org>

Subject: grafana: FTBFS when built with dpkg-buildpackage -A (unable to chdir to _build)

Date: Tue, 08 Mar 2016 10:02:46 +0000

Package: src:grafana
Version: 2.6.0+dfsg-3
User: sanvila@debian.org
Usertags: binary-indep
Severity: important

Dear maintainer:

I tried to build this package with "dpkg-buildpackage -A"
(i.e. only architecture-independent packages), and it failed:

--------------------------------------------------------------------------------
[...]
 debian/rules build-indep
## Build CSS: http://lesscss.org/usage/index.html
mkdir -p public/css
cd public/css \
        && lessc --include-path=../less --include-path=../vendor/bootstrap/less ../less/bootstrap.dark.less grafana.dark.min.css.tmp \
        && cat grafana.dark.min.css.tmp /usr/share/fonts-font-awesome/css/font-awesome.css > grafana.dark.min.css \
        && lessc --include-path=../less --include-path=../vendor/bootstrap/less ../less/bootstrap.light.less grafana.light.min.css.tmp \
        && cat grafana.light.min.css.tmp /usr/share/fonts-font-awesome/css/font-awesome.css > grafana.light.min.css \
        ;
tsc --module amd public/test/lib/common.ts public/test/core/utils/rangeutil_specs.ts public/test/core/utils/datemath_specs.ts public/test/core/utils/flatten_specs.ts public/test/core/table_model_specs.ts public/app/headers/lodash/lodash.d.ts public/app/headers/moment/moment.d.ts public/app/headers/moment/moment-node.d.ts public/app/headers/angularjs/angularjs.d.ts public/app/headers/common.d.ts public/app/headers/jquery/jquery.d.ts public/app/headers/require/require.d.ts public/app/features/panel/panel_meta.ts public/app/features/dashboard/timepicker/timepicker.ts public/app/plugins/datasource/elasticsearch/specs/index_pattern_specs.ts public/app/plugins/datasource/elasticsearch/specs/query_ctrl_specs.ts public/app/plugins/datasource/elasticsearch/specs/query_def_specs.ts public/app/plugins/datasource/elasticsearch/specs/elastic_response_specs.ts public/app/plugins/datasource/elasticsearch/specs/query_builder_specs.ts public/app/plugins/datasource/elasticsearch/specs/datasource_specs.ts public/app/plugins/datasource/graphite/specs/gfunc_specs.ts public/app/plugins/datasource/graphite/specs/query_ctrl_specs.ts public/app/plugins/datasource/graphite/specs/datasource_specs.ts public/app/plugins/datasource/cloudwatch/specs/datasource_specs.ts public/app/plugins/datasource/prometheus/specs/datasource_specs.ts public/app/plugins/datasource/influxdb/influx_query.ts public/app/plugins/datasource/influxdb/specs/query_ctrl_specs.ts public/app/plugins/datasource/influxdb/specs/influx_series_specs.ts public/app/plugins/datasource/influxdb/specs/query_builder_specs.ts public/app/plugins/datasource/influxdb/specs/query_part_specs.ts public/app/plugins/datasource/influxdb/specs/influx_query_specs.ts public/app/plugins/datasource/influxdb/query_part.ts public/app/plugins/datasource/influxdb_08/specs/influx_series_specs.ts public/app/plugins/datasource/influxdb_08/specs/query_builder_specs.ts public/app/plugins/datasource/influxdb_08/specs/datasource-specs.ts public/app/core/time_series.ts public/app/core/directives/give_focus.ts public/app/core/directives/array_join.ts public/app/core/utils/rangeutil.ts public/app/core/utils/datemath.ts public/app/core/utils/flatten.ts public/app/core/controllers/signup_ctrl.ts public/app/core/core.ts public/app/core/routes/bundle_loader.ts public/app/core/table_model.ts public/app/core/filters/filters.ts public/app/core/core_module.ts public/app/panels/table/module.ts public/app/panels/table/editor.ts public/app/panels/table/specs/renderer_specs.ts public/app/panels/table/specs/table_model_specs.ts public/app/panels/table/specs/transformers_specs.ts public/app/panels/table/controller.ts public/app/panels/table/renderer.ts public/app/panels/table/transformers.ts
 fakeroot debian/rules binary-indep
dh binary-indep --buildsystem=golang --with=golang,systemd --builddirectory=_build --parallel
   debian/rules build-indep
make[1]: Entering directory '/<<BUILDDIR>>/grafana-2.6.0+dfsg'
## Build CSS: http://lesscss.org/usage/index.html
mkdir -p public/css
cd public/css \
        && lessc --include-path=../less --include-path=../vendor/bootstrap/less ../less/bootstrap.dark.less grafana.dark.min.css.tmp \
        && cat grafana.dark.min.css.tmp /usr/share/fonts-font-awesome/css/font-awesome.css > grafana.dark.min.css \
        && lessc --include-path=../less --include-path=../vendor/bootstrap/less ../less/bootstrap.light.less grafana.light.min.css.tmp \
        && cat grafana.light.min.css.tmp /usr/share/fonts-font-awesome/css/font-awesome.css > grafana.light.min.css \
        ;
tsc --module amd public/test/lib/common.ts public/test/core/utils/rangeutil_specs.ts public/test/core/utils/datemath_specs.ts public/test/core/utils/flatten_specs.ts public/test/core/table_model_specs.ts public/app/headers/lodash/lodash.d.ts public/app/headers/moment/moment.d.ts public/app/headers/moment/moment-node.d.ts public/app/headers/angularjs/angularjs.d.ts public/app/headers/common.d.ts public/app/headers/jquery/jquery.d.ts public/app/headers/require/require.d.ts public/app/features/panel/panel_meta.ts public/app/features/dashboard/timepicker/timepicker.ts public/app/plugins/datasource/elasticsearch/specs/index_pattern_specs.ts public/app/plugins/datasource/elasticsearch/specs/query_ctrl_specs.ts public/app/plugins/datasource/elasticsearch/specs/query_def_specs.ts public/app/plugins/datasource/elasticsearch/specs/elastic_response_specs.ts public/app/plugins/datasource/elasticsearch/specs/query_builder_specs.ts public/app/plugins/datasource/elasticsearch/specs/datasource_specs.ts public/app/plugins/datasource/graphite/specs/gfunc_specs.ts public/app/plugins/datasource/graphite/specs/query_ctrl_specs.ts public/app/plugins/datasource/graphite/specs/datasource_specs.ts public/app/plugins/datasource/cloudwatch/specs/datasource_specs.ts public/app/plugins/datasource/prometheus/specs/datasource_specs.ts public/app/plugins/datasource/influxdb/influx_query.ts public/app/plugins/datasource/influxdb/specs/query_ctrl_specs.ts public/app/plugins/datasource/influxdb/specs/influx_series_specs.ts public/app/plugins/datasource/influxdb/specs/query_builder_specs.ts public/app/plugins/datasource/influxdb/specs/query_part_specs.ts public/app/plugins/datasource/influxdb/specs/influx_query_specs.ts public/app/plugins/datasource/influxdb/query_part.ts public/app/plugins/datasource/influxdb_08/specs/influx_series_specs.ts public/app/plugins/datasource/influxdb_08/specs/query_builder_specs.ts public/app/plugins/datasource/influxdb_08/specs/datasource-specs.ts public/app/core/time_series.ts public/app/core/directives/give_focus.ts public/app/core/directives/array_join.ts public/app/core/utils/rangeutil.ts public/app/core/utils/datemath.ts public/app/core/utils/flatten.ts public/app/core/controllers/signup_ctrl.ts public/app/core/core.ts public/app/core/routes/bundle_loader.ts public/app/core/table_model.ts public/app/core/filters/filters.ts public/app/core/core_module.ts public/app/panels/table/module.ts public/app/panels/table/editor.ts public/app/panels/table/specs/renderer_specs.ts public/app/panels/table/specs/table_model_specs.ts public/app/panels/table/specs/transformers_specs.ts public/app/panels/table/controller.ts public/app/panels/table/renderer.ts public/app/panels/table/transformers.ts
make[1]: Leaving directory '/<<BUILDDIR>>/grafana-2.6.0+dfsg'
   dh_testroot -i -O--buildsystem=golang -O--builddirectory=_build -O--parallel
   dh_prep -i -O--buildsystem=golang -O--builddirectory=_build -O--parallel
   dh_auto_install -i -O--buildsystem=golang -O--builddirectory=_build -O--parallel
dh_auto_install: error: unable to chdir to _build
debian/rules:15: recipe for target 'binary-indep' failed
make: *** [binary-indep] Error 2
dpkg-buildpackage: error: fakeroot debian/rules binary-indep gave error exit status 2
--------------------------------------------------------------------------------

Sorry not to have a fix, as I am reporting many bugs similar to
this one. The common hints are:

* If the only architecture-independent packages are dummy transitional
ones and they were released with jessie, the easy fix is to drop them
now.

* When using "dh", it is allowed to use (independently)
optional targets override_dh_foo-arch and override_dh_foo-indep
(for several values of "foo").


Once that both "dpkg-buildpackage -A" and "dpkg-buildpackage -B" work
properly, the package would be suitable to be uploaded in source-only
form if you wish.

Thanks.

Message #10 received at submit@bugs.debian.org (full text, mbox, reply):

From: Dmitry Smirnov <onlyjob@debian.org>

To: Santiago Vila <sanvila@debian.org>, 817126@bugs.debian.org

Cc: Debian BTS <submit@bugs.debian.org>

Subject: Re: Bug#817126: grafana: FTBFS when built with dpkg-buildpackage -A (unable to chdir to _build)

Date: Tue, 08 Mar 2016 23:41:13 +1100

[Message part 1 (text/plain, inline)]

On Tuesday, 8 March 2016 10:02:46 AM AEDT Santiago Vila wrote:
> -O--parallel dh_auto_install -i -O--buildsystem=golang
> -O--builddirectory=_build -O--parallel dh_auto_install: error: unable to
> chdir to _build

This is not a problem in Grafana. I think the error is in Debhelper
buildsystem=golang module (dh-golang). dh_auto_install call fails when
builddirectory do not exist...

-- 
Cheers,
 Dmitry Smirnov.

---

The Santa myth is one of the most effective means ever devised for
intimidating children, eroding their self-esteem, twisting their
behavior, warping their values, and slowing their development of
critical thinking skills.
        -- Tom Flynn, "The Trouble with Christmas"

[signature.asc (application/pgp-signature, inline)]

Message #22 received at 817126@bugs.debian.org (full text, mbox, reply):

From: Santiago Vila <sanvila@unex.es>

To: Dmitry Smirnov <onlyjob@debian.org>

Cc: 817126@bugs.debian.org

Subject: Re: Bug#817126: grafana: FTBFS when built with dpkg-buildpackage -A (unable to chdir to _build)

Date: Tue, 8 Mar 2016 14:36:11 +0100

On Tue, Mar 08, 2016 at 11:41:13PM +1100, Dmitry Smirnov wrote:
> On Tuesday, 8 March 2016 10:02:46 AM AEDT Santiago Vila wrote:
> > -O--parallel dh_auto_install -i -O--buildsystem=golang
> > -O--builddirectory=_build -O--parallel dh_auto_install: error: unable to
> > chdir to _build
> 
> This is not a problem in Grafana. I think the error is in Debhelper
> buildsystem=golang module (dh-golang). dh_auto_install call fails when
> builddirectory do not exist...

This is hardly a problem in debhelper, which is just a tool to
simplify debian/rules. What debhelper does is what you tell it to do,
nothing less and nothing more.

In this case we are using "dpkg-buildpackage -A" so the generic dh rule
does not work for the "build" because there is a specific build-indep target.

So it's not surprising that the build directory does not exist.

If you have to split dh_auto_install into -arch and -indep, go ahead,
but please do not blame debhelper.

Thanks.

Message #31 received at 817126-submitter@bugs.debian.org (full text, mbox, reply):

From: Dmitry Smirnov <onlyjob@member.fsf.org>

To: 817126-submitter@bugs.debian.org

Subject: Bug#817126 tagged as pending

Date: Tue, 05 Apr 2016 03:50:15 +0000

tag 817126 pending
--

We believe that the bug #817126 you reported has been fixed in the Git
repository. You can see the commit message below and/or inspect the
commit contents at:

    http://anonscm.debian.org/cgit/collab-maint/grafana.git/diff/?id=0cd0d66

(This message was generated automatically by
 'git-post-receive-tag-pending-commitmsg' hook).
---
commit 0cd0d66 (HEAD, master)
Author: Dmitry Smirnov <onlyjob@member.fsf.org>
Date:   Tue Apr 5 03:45:54 2016

    Fixed FTBFS with dpkg-buildpackage -A (Closes: #817126)

Message #36 received at 817126@bugs.debian.org (full text, mbox, reply):

From: Santiago Vila <sanvila@unex.es>

To: Dmitry Smirnov <onlyjob@member.fsf.org>

Cc: 817126@bugs.debian.org

Subject: Re: Bug#817126 tagged as pending

Date: Wed, 13 Jul 2016 22:44:39 +0200 (CEST)

Greetings.

I have the ok from the Release Managers to consider this issue
as RC for stretch.

I'm going to wait a week before raising severities, but since this bug
is in pending state, I'd appreciate if you could make an upload
before that.

Thanks.

Message #41 received at 817126@bugs.debian.org (full text, mbox, reply):

From: Dmitry Smirnov <onlyjob@debian.org>

To: Santiago Vila <sanvila@unex.es>, 817126@bugs.debian.org

Subject: Re: Bug#817126: grafana: FTBFS when built with dpkg-buildpackage -A

Date: Thu, 14 Jul 2016 15:35:26 +1000

[Message part 1 (text/plain, inline)]

On Wednesday, 13 July 2016 10:44:39 PM AEST Santiago Vila wrote:
> I have the ok from the Release Managers to consider this issue
> as RC for stretch.

I think it is not a good idea. IMHO severity "important" was perfect for 
those issues. Bumping severity to "serious" would force maintainers to 
prioritise over issues that have direct impact on users. Nobody likes bugs 
with inflated severity...


> I'm going to wait a week before raising severities, but since this bug
> is in pending state, I'd appreciate if you could make an upload
> before that.

Unfortunately I can't upload due to another problem (#828814) that prevents 
successful build of grafana...

-- 
All the best,
 Dmitry Smirnov
 GPG key : 4096R/52B6BBD953968D1B

---

However beautiful the strategy, you should occasionally look at the
results.
        -- Winston Churchill

[signature.asc (application/pgp-signature, inline)]

Message #46 received at 817126@bugs.debian.org (full text, mbox, reply):

From: Santiago Vila <sanvila@unex.es>

To: Dmitry Smirnov <onlyjob@debian.org>

Cc: 817126@bugs.debian.org

Subject: Re: Bug#817126: grafana: FTBFS when built with dpkg-buildpackage -A

Date: Thu, 14 Jul 2016 12:31:08 +0200 (CEST)

On Thu, 14 Jul 2016, Dmitry Smirnov wrote:

> On Wednesday, 13 July 2016 10:44:39 PM AEST Santiago Vila wrote:
> > I have the ok from the Release Managers to consider this issue
> > as RC for stretch.
> 
> I think it is not a good idea. IMHO severity "important" was perfect for 
> those issues. Bumping severity to "serious" would force maintainers to 
> prioritise over issues that have direct impact on users. Nobody likes bugs 
> with inflated severity...

Well, bear in mind that this has always been a release goal: "packages
must autobuild". The only difference is that in the past we didn't
have an "Arch: all" autobuilder, and now we do.

The idea here (in addition to being able to make source-only uploads)
is that the "Arch: all" autobuilder should not be a second-class
citizen among all autobuilders.

> > I'm going to wait a week before raising severities, but since this bug
> > is in pending state, I'd appreciate if you could make an upload
> > before that.
> 
> Unfortunately I can't upload due to another problem (#828814) that prevents 
> successful build of grafana...

Ok, don't worry. My suggestion to make an upload was only to clean up
the list of bugs a little bit.

This is (or will be) just a release goal for stretch, not a security
bug in stable.

Since you are alive and not MIA, and the bug is fixed in git, that's
more than enough for now.

Thanks.

Send a report that this bug log contains spam.