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

Bug#762933: marked as done (debsources: webapp should not require python-matplotlib)

Your message dated Tue, 24 Mar 2015 17:08:54 +0100
with message-id <20150324160854.GA726@upsilon.cc>
and subject line Re: Bug#762933: debsources: webapp should not require python-matplotlib
has caused the Debian Bug report #762933,
regarding debsources: webapp should not require python-matplotlib
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
762933: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=762933
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: qa.debian.org
Severity: normal
User: qa.debian.org@packages.debian.org
Usertags: debsources

Since the refactoring with the top-level debsources/ module, the webapp
requires python-matplotlib to be installed to run properly. That is
unfortunate, as matplotlib is needed only by the updater, and induces weird
requirements on the webserver user (e.g., a writable /var/www/.matplotlib ?!!).

We should track down and remove this dependency.

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (500, 'testing'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.14-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=it_IT.UTF-8, LC_CTYPE=it_IT.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

--- End Message ---
--- Begin Message --- 
On Tue, Mar 24, 2015 at 02:05:42PM +0100, Clément Schreiner wrote:
> The dependency comes from debsources.mainlib, which imports
> debsources.updater, which imports debsources.charts.

Nice catch, thanks!
Patch integrated.

I agree with you that it is worth drifting from PEP8 in this case, and
I've elaborated a bit on why in the (re-worded) commit log.

Oddly enough, flake8 does not complain about this issue though, so I've
refrained from adding "# NOQA".

Closing the bug,
Cheers.
-- 
Stefano Zacchiroli  . . . . . . .  zack@upsilon.cc . . . . o . . . o . o
Maître de conférences . . . . . http://upsilon.cc/zack . . . o . . . o o
Former Debian Project Leader  . . @zack on identi.ca . . o o o . . . o .
« the first rule of tautology club is the first rule of tautology club »

Attachment: signature.asc
Description: Digital signature


--- End Message ---
Reply to: