Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Hi, sorry for the delay.

On Sat, Jan 18, 2020 at 7:57 PM Samuel Henrique  wrote:
>
> Hello Jason,
>
> > Feel free to take over the ITP and the packaging. I had deleted the
> > repo because I wanted to start over when I decided not to split the
> > libraries and I forgot to re-create and push what I had so far. I'll
> > try to have it on salsa over the weekend.
>
> Great, I managed to get a working deb package, but there are still some things
> like d/copyright and documentation fixes missing.
>
> After I get your changes I will merge them together and then finish what's
> missing. I'm aiming to get it in the NEW queue in a week, so hopefully it
> will get accepted before the end of February (and it can hit Ubuntu 20.04).
>
> If it's of any concern for you, I don't mind about how you send/publish your
> work or how polished it is, you can just send me the debian folder if it's 
> easier
> for you, I'm sure it will be useful anyway.
>
> Regards,
>

I pushed what I had here: https://gitlab.com/jpleau/polybar/

It's not targetting debian (simply because I'm using ubuntu these
days). Feel free to take whatever you find useful

Thanks for taking overe !
> --
> Samuel Henrique 

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Hi

On Sat, Jan 11, 2020 at 5:08 PM Samuel Henrique  wrote:
>
> Hello Jason,
>
> I'm interested in having polybar packaged on Debian,
>
> I can see that you closed the ITP of the other two libs libxpp and i3ipcpp
> stating that they are no longer needed, and that the repository for polybar's
> packaging on salsa is not available anymore.
>
> Do you have update0s on its packaging? I'd be happy to help or takeover from
> where you stopped (if you have given up).
>
> From reading the discussion, I'm tempted to not split polybar's libs into 
> different
> packages. I assume that's the current direction you're going as well.
>

Feel free to take over the ITP and the packaging. I had deleted the
repo because I wanted to start over when I decided not to split the
libraries and I forgot to re-create and push what I had so far. I'll
try to have it on salsa over the weekend.
>
> Regards,
>
> --
> Samuel Henrique 

Bug#928664: lua-system has a dependency cycle with lua-busted

[Jason Pleau]

Hi

On Wed, May 8, 2019 at 10:27 AM Helmut Grohne  wrote:
>
> Source: lua-system
> Version: 0.2.1-1
> Severity: important
> Tags: patch
> User: helm...@debian.org
> Usertags: rebootstrap
>
> lua-system Build-Depends on lua-busted, which happens to depend on
> lua-system. This poses a dependency cycle and makes bootstrapping either
> impossible. The upshot is that lua-system only needs lua-busted for
> running unit tests. One can build lua-system with
> DEB_BUILD_OPTIONS=nocheck, then lua-busted and then the full lua-system.
> Easy. All we need to do here is annotate the dependency. Please consider
> applying the attached patch.
>
> Helmut


I'll be able to look at this in the coming days (maybe next week),
feel free to do an NMU in the meantime if I can't get to it fast
enough.

Thanks !

Bug#923662: mark lua-say Multi-Arch: foreign

[Jason Pleau]

Hi

On 2019-03-03 7:35 a.m., Helmut Grohne wrote:
> Source: lua-say
> Version: 1.3-1-4
> Tags: patch
> User: debian-cr...@lists.debian.org
> Usertags: cross-satisfiability
> Control: affects -1 + src:prosody
> 
> prosody fails to cross build from source, because its transitive
> dependency on lua-say is unsatisfiable. It seems that marking lua-say
> Multi-Arch: foreign is part of the solution here. It is an
> architecture-independent package with no dependencies or maintainer
> scripts. That usually means that marking it Multi-Arch: foreign is safe.
> Please consider applying the attached patch.

The repo has been updated: https://salsa.debian.org/lua-team/lua-say

Do you think this warrants an unblock request? I can arrange for an
upload, but will request an unblock if this blocks another package from
working correctly.

Thanks.

> 
> Helmut
> 

-- 
Jason Pleau

Bug#923909: unblock: lua-nvim/0.1.0-1-2

[Jason Pleau]

Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock

Please unblock package lua-nvim

This version fixes a FTBFS (#920042) in a test during autopkgtest.

Defdiff attached. thanks.

unblock lua-nvim/0.1.0-1-2
diff -Nru lua-nvim-0.1.0-1/debian/changelog lua-nvim-0.1.0-1/debian/changelog
--- lua-nvim-0.1.0-1/debian/changelog   2018-06-11 20:44:29.0 -0400
+++ lua-nvim-0.1.0-1/debian/changelog   2019-03-04 21:12:53.0 -0500
@@ -1,3 +1,10 @@
+lua-nvim (0.1.0-1-2) unstable; urgency=medium
+
+  * New changelog entry
+  * Include a patch to fix a FTBFS due to test timeout (Closes: #920042)
+
+ -- Jason Pleau   Mon, 04 Mar 2019 21:12:53 -0500
+
 lua-nvim (0.1.0-1-1) unstable; urgency=medium
 
   * New upstream release
diff -Nru 
lua-nvim-0.1.0-1/debian/patches/0001-Use-embed-headless-for-Socket-TcpStream-session-test.patch
 
lua-nvim-0.1.0-1/debian/patches/0001-Use-embed-headless-for-Socket-TcpStream-session-test.patch
--- 
lua-nvim-0.1.0-1/debian/patches/0001-Use-embed-headless-for-Socket-TcpStream-session-test.patch
 1969-12-31 19:00:00.0 -0500
+++ 
lua-nvim-0.1.0-1/debian/patches/0001-Use-embed-headless-for-Socket-TcpStream-session-test.patch
 2019-03-04 21:12:53.0 -0500
@@ -0,0 +1,36 @@
+From 24fe448aa406b0aa2267044a9858e3d2e5963b61 Mon Sep 17 00:00:00 2001
+From: James McCoy 
+Date: Sun, 3 Mar 2019 11:49:54 -0500
+Subject: [PATCH] Use --embed --headless for Socket/TcpStream session tests
+
+After neovim/neovim#9024, "nvim --embed" waits for a UI to attach.
+"nvim --embed --headless" is required if there is not going to be a UI.
+---
+ test/session_spec.lua | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/test/session_spec.lua b/test/session_spec.lua
+index d5d705c..1fc289d 100644
+--- a/test/session_spec.lua
 b/test/session_spec.lua
+@@ -155,7 +155,7 @@ end)
+ -- Session using SocketStream
+ test_session(string.format("Session using SocketStream [%s]", socket_file), 
function ()
+   child_session = Session.new(ChildProcessStream.spawn({
+-nvim_prog, '-u', 'NONE', '--embed',
++nvim_prog, '-u', 'NONE', '--embed', '--headless',
+ '--cmd', string.format('call serverstart("%s")', socket_file)
+   }))
+   child_session:request('vim_eval', '1') -- wait for nvim to start
+@@ -193,7 +193,7 @@ end)
+ -- Session using TcpStream
+ test_session("Session using TcpStream", function ()
+   child_session = Session.new(ChildProcessStream.spawn({
+-nvim_prog, '-u', 'NONE', '--embed',
++nvim_prog, '-u', 'NONE', '--embed', '--headless',
+ '--cmd', 'call serverstart("127.0.0.1:")'
+   }))
+ 
+-- 
+2.20.1
+
diff -Nru lua-nvim-0.1.0-1/debian/patches/series 
lua-nvim-0.1.0-1/debian/patches/series
--- lua-nvim-0.1.0-1/debian/patches/series  2018-06-11 20:44:29.0 
-0400
+++ lua-nvim-0.1.0-1/debian/patches/series  2019-03-04 21:12:53.0 
-0500
@@ -0,0 +1 @@
+0001-Use-embed-headless-for-Socket-TcpStream-session-test.patch

Bug#923741: new upstream release

[Jason Pleau]

Hi

On 2019-03-04 4:53 p.m., Antoine Beaupre wrote:
> Package: rofi
> Version: 1.5.1-1
> Severity: wishlist
> 
> A new release (1.5.2) of rofi was published:
> 
> https://github.com/DaveDavenport/rofi/releases/tag/1.5.2
> 
> I've tried to upgrade this myself but there's weird stuff going on, as
> "git diff 1.5.0..upstream/1.5.0" differs, yet there's no +dfsg tag or
> a clear way to reproduce the orig tarball from the git repo.
> 

I don't use the github-generated tarballs, they are missing certain
files (an active contributor to rofi has different projects that aren't
meant to be distributed separately, but are included in rofi. Those are
pulled in when they run 'make dist' iirc)


I used gbp import-orig --upstream-vcs-tag=1.5.1 rofi-1.5.1.tar.gz (file
taken from
https://github.com/DaveDavenport/rofi/releases/download/1.5.2/rofi-1.5.1.tar.gz,
different than the github tag tarball).

Hope that helps. Thanks for looking into the update, I've been busy with
other stuff for a little while now =/


> -- System Information:
> Debian Release: buster/sid
>   APT prefers testing
>   APT policy: (500, 'testing'), (1, 'experimental'), (1, 'unstable')
> Architecture: amd64 (x86_64)
> 
> Kernel: Linux 4.19.0-2-amd64 (SMP w/4 CPU cores)
> Kernel taint flags: TAINT_OOT_MODULE, TAINT_UNSIGNED_MODULE
> Locale: LANG=fr_CA.UTF-8, LC_CTYPE=fr_CA.UTF-8 (charmap=UTF-8), 
> LANGUAGE=fr_CA.UTF-8 (charmap=UTF-8)
> Shell: /bin/sh linked to /usr/bin/dash
> Init: systemd (via /run/systemd/system)
> LSM: AppArmor: enabled
> 
> Versions of packages rofi depends on:
> ii  libc6 2.28-7
> ii  libcairo2 1.16.0-2
> ii  libglib2.0-0  2.58.3-1
> ii  libpango-1.0-01.42.4-6
> ii  libpangocairo-1.0-0   1.42.4-6
> ii  librsvg2-22.44.10-1
> ii  libstartup-notification0  0.12-6
> ii  libxcb-ewmh2  0.4.1-1.1
> ii  libxcb-icccm4 0.4.1-1.1
> ii  libxcb-randr0 1.13.1-2
> ii  libxcb-util0  0.3.8-3+b2
> ii  libxcb-xinerama0  1.13.1-2
> ii  libxcb-xkb1   1.13.1-2
> ii  libxcb-xrm0   1.0-3
> ii  libxcb1   1.13.1-2
> ii  libxkbcommon-x11-00.8.2-1
> ii  libxkbcommon0 0.8.2-1
> 
> rofi recommends no packages.
> 
> rofi suggests no packages.
> 
> -- no debconf information
> 

-- 
Jason Pleau

Bug#894536: vim: Name suggestions for edit command should not include directories

[Jason Pleau]

Hi

On 04/01/2018 01:27 AM, David Christensen wrote:
> Package: vim
> Version: 2:8.0.0197-4+deb9u1
> Severity: wishlist
> 
> Dear Maintainer,
> 
> The 'e' (edit) command of Vim has command-line completion.  Items
> offered include file names and directory names.  Directory names
> should not be included by default.
> 
> TIA,
> 
> David
> 

I think this is the correct behavior and should stay default.

Typing :e to edit this file:

/some/verylongdirectorynames/tohideaveryspecial/file

I am very grateful to have autocompletion :)


-- 
Jason Pleau

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Small update,

I pushed a change to i3ipcpp that also builds a shared library in
addition to the static library. (The i3ipcpp patches I will probably be
able to send upstream at some point). polybar correctly uses the shared
library, I think it's better this way.

I don't think I can do the same with xpp, since it is a header only
library and a shared library would probably be pointless. I think I can
even remove the static library (libxpp.a); it's empty. The static
linkage happens with the xcb libraries anyway, not with xpp.


-- 
Jason Pleau

Bug#893520: ImportError: No module named backports.ssl_match_hostname

[Jason Pleau]

Small typo in my previous email, the .prerm script contains:

pyclean -p python-backports.ssl-match-hostname

I had another package in my clipboard, just wanted to clear that up so
there's no confusion :)

-- 
Jason Pleau

Bug#893520: ImportError: No module named backports.ssl_match_hostname

[Jason Pleau]

Hi. I was also affected by this bug and I managed to reproduce it.

> Why do you think that the file going missing is a python-minimal bug?

The missing file seems to be generated by the postinst script which
contains:

pycompile -p python-backports.ssl-match-hostname


But, there's also a .prerm script for this package that runs this:

pyclean -p python-backports-shutil-get-terminal-size


If you install (reinstall) those packages:

python-backports.ssl-match-hostname
python-backports-shutil-get-terminal-size

And remove either of them, you'll end up with

/usr/lib/python2.7/dist-packages/backports/__init__.py{c}

missing.


Maybe those scripts should check if those files belong to another
packages before running pyclean

There are other python-backports-* packages that probably suffer from
the same problem.


-- 
Jason Pleau

Bug#894413: ITP: siji -- iconic bitmap font to use on status bars

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: siji
  Version : git (no release)
  Upstream Author : stark <thestarkni...@tutanota.com>
* URL : https://github.com/stark/siji/
* License : GPL-2
  Programming Lang:
  Description : iconic bitmap font to use on status bars

siji is a fork of the no longer maintained "Stlarch" fonts, with
additional glyphs.

It contains most icons one would like to display on a status bar.

siji is a dependency of "polybar" (ITP #856179). It is used in the
default config upstream uses, I would like to stay as close as possible
from that default config.

I intend to maintain this package with the Debian Fonts Task Force team.

-- 
Jason Pleau

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Hi (again..!)

Didn't expect to get something to work so soon regarding the splitting
of the packages.

On salsa:

https://salsa.debian.org/jpleau-guest/polybar (BRANCH: split_pkg)
https://salsa.debian.org/jpleau-guest/libxpp
https://salsa.debian.org/jpleau-guest/i3ipcpp

In my sid chroot, all 3 now build fine, and the installed package works
on my machine (you'll probably find the same issues you did before with
the config and fonts though, haven't worked those yet)

I have a patch that I will have to carry for polybar, which includes
bits of xpp's CMake file to include the necessary XCB include /
libraries (for linking). I think that's a bit less ugly than carrying
the whole thing together :)

Cheers

-- 
Jason Pleau

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Hi,

I'll try to answer your questions :)

On 03/28/2018 10:51 AM, Antoine Beaupré wrote:
> On 2018-03-27 21:15:35, Jason Pleau wrote:
>> Hi.
>>
>> I took a few hours last weekend to work on this.
> 
> Awesome, thanks for the work!
> 
>> While I was able to have "working" packages for both xpp and i3ipcpp,
>> I could not get polybar to use them (the whole thing is glued together
>> nicely it seems and trying to split them caused me headaches). So I
>> went ahead and worked on packaging the whole repo (and submodules)
>> together.
> 
> Can you expand on the problems you've encountered?

Sure. Basically, by itself xpp, it's a header-only library. To build a
project using it you have to include xpp's CMakeLists.txt from cmake,
which then includes other cmake modules to find the different XCB modules.

xpp's CMake file then generates the appropriate XCB_* libraries for your
own project (in this case, polybar) to link against.

So, if I was to take "xpp" into it's own package, polybar currently
would have no way to know which XCB libraries to link against, since it
uses xpp's own CMakeFile to find those. That's where I was stuck (I
don't really use CMake so this stuff is out of my reach for the time being)

> 
>> Repo: https://salsa.debian.org/jpleau-guest/polybar
>>
>> Current status: it builds in a chroot and works on my sid install.
> 
> I have tried to build this in stretch and failed:
> 
> $ sbuild -c stretch
> dh clean --buildsystem=cmake --builddirectory=build
>dh_auto_clean -O--buildsystem=cmake -O--builddirectory=build
>dh_autoreconf_clean -O--buildsystem=cmake -O--builddirectory=build
>dh_clean -O--buildsystem=cmake -O--builddirectory=build
> dpkg-source: error: can't build with source format '3.0 (quilt)': no upstream 
> tarball found at ../polybar_3.1.0.orig.tar.{bz2,gz,lzma,xz}
> E: Failed to package source directory /home/anarcat/dist/polybar
> 1$ uscan
> uscan warn: No watch file found
> 1$ gbp buildpackage -c stretch
> dh clean --buildsystem=cmake --builddirectory=build
>dh_auto_clean -O--buildsystem=cmake -O--builddirectory=build
>dh_autoreconf_clean -O--buildsystem=cmake -O--builddirectory=build
>dh_clean -O--buildsystem=cmake -O--builddirectory=build
> gbp:error: upstream/3.1.0 is not a valid treeish
> 
> So a few things here:
> 
>  * a debian/watch file would be useful, even if just to find out new
>versions are coming out...
>  * the upstream tree should be tagged
>  
> When those are fixed, I get this:
> 
>  sbuild-build-depends-polybar-dummy : Depends: debhelper (>= 11) but it is 
> not going to be installed
> 
> So it might also be useful to make the DH dependency >= 11~ to allow for
> easier backporting. I can send a merge request for that on Salsa (or a
> patch here) if you want.
> 

1) watch file: I can add the watch file, if only to check for new versions.

2) debhelper version: sure, I didn't really test building for stretch
but I see no harm in changing the version for debhelper if it means
polybar can be backported (no need to do a MR I'll take care of it)

>> TODO:
>>
>> - There's a few copyright info missing (ie: lib/concurrentqueue)-
> 
> Seems to be 2-clause BSD.

Yep, I saw it but I didn't add the changes yet to debian/copyright
(hence the TODO title!)

> 
>> - After installing the package, it won't do anything because the config
>> file is not found (it should be in $HOME/.config/polybar). There is one
>> shipped in /usr/share/doc/polybar/config.gz, but surely there's a way to
>> tell the users that when they install the package?
> 
> /usr/share/doc/polybar/README.Debian is usually where I would expect
> that kind of information to be, or in the manpage, or in the error
> message directly.. Also, I would expect to find the config.gz file in an
> examples/ subdirectory there.
> 
> Maybe a more long-term solution would be to ship the sample config file
> in /etc/polybar/config and patch the package to look for there on top of
> $XDG_CONFIG_HOME. The proper place to look for those is XDG_CONFIG_DIRS,
> which defaults to /etc/xdg, which I've always found weird. See this spec
> for details:
> 
> https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html

Yes I think upstream would be in favor of having a "system-wide" config
file to use as fallback. /etc/polybar or /etc/polybar/config, I'll have
to see what other programs do.
> 
>> Note that I made a custom get-orig-source rule. The tarball didn't
>> contain xpp and i3ipcpp (github generated tarballs don't include
>> submodules). It seems to work fine, feedback welcome on this one..
> 
> hmm... that does look kind of nasty. :p Why is the vers

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Hi.

I took a few hours last weekend to work on this. While I was able to
have "working" packages for both xpp and i3ipcpp, I could not get
polybar to use them (the whole thing is glued together nicely it seems
and trying to split them caused me headaches).

So I went ahead and worked on packaging the whole repo (and submodules)
together.

Repo: https://salsa.debian.org/jpleau-guest/polybar

Current status: it builds in a chroot and works on my sid install.

TODO:

- There's a few copyright info missing (ie: lib/concurrentqueue)-
- After installing the package, it won't do anything because the config
file is not found (it should be in $HOME/.config/polybar). There is one
shipped in /usr/share/doc/polybar/config.gz, but surely there's a way to
tell the users that when they install the package?

Note that I made a custom get-orig-source rule. The tarball didn't
contain xpp and i3ipcpp (github generated tarballs don't include
submodules). It seems to work fine, feedback welcome on this one..

Thanks

-- 
Jason Pleau

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Hello.

On 02/24/2018 11:11 AM, Antoine Beaupré wrote:
> On 2018-02-23 22:47:08, Jason Pleau wrote:
>> Hi.
>> [...]

>> i3ipcpp (github.com/jaagr/i3ipcpp, forked from drmgc/i3pcpp)
>>   - auss (github.com/jaagr/auss, forked from drmgc/auss)
>>   - jsoncpp (seems to be in Debian as src:libjsoncpp)
>> xpp (github.com/jaagr/xpp, forked from jotrk/xpp)
>>
>>
>> I don't mind doing the work of packaging these 3 libraries, however I
>> would package the forks, unless someone wants to step in and try to get
>> the forks merged into their original upstreams.
> 
> Makes sense. Should we file RFPs for those or will you Just Do It? :)

I just filed the ITPs (#892007, #892008). I have working packages for
both on my machine here, just need to iron out a few details.

>> Of course, I would welcome any help with this.
> 
> I don't have much time, but I would gladly test packages and I can
> sponsor/review uploads.
> 

Awesome thanks. Will let you know when I have something ready!

> A.
> 

-- 
Jason Pleau

Bug#892008: ITP: i3ipcpp -- C++ interface to i3-ipc

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: i3ipcpp
  Version : 0.7.1
  Upstream Author : Sergey Naumov <dr...@yandex.ru>
* URL : https://github.com/jaagr/i3ipcpp
* License : MIT
  Programming Lang: C++
  Description : C++ interface to i3-ipc

Window manager i3-wm ships with an IPC interface (interprocess communication),
accessible through a CLI utility named "i3-ipc".

i3ipcpp provides a way to use this IPC interface through C++ code.

This package is a dependency of polybar (ITP #856179)

Bug#892007: ITP: libxpp -- A C++11 RAII wrapper for XCB

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: libxpp
  Version : 1.4.0
  Upstream Author : Jochen Keil <jochen.k...@gmail.com>
* URL : https://github.com/jaagr/xpp
* License : MIT
  Programming Lang: C++, Python
  Description : A C++11 RAII wrapper for XCB

XPP is a header only C++11 RAII wrapper around X protocol C-language Binding
(XCB).

Since there is another package named "xpp" in Debian, I am going to use
"libxpp" as the source package name.

Note that I am packaging a fork of xpp. The original repository hasn't been
updated in 4 years, the features that polybar requires were added in the fork.

This package is a dependency of polybar (ITP #856179)

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Hi.

On 02/20/2018 01:51 PM, Antoine Beaupre wrote:
> On Sat, Feb 25, 2017 at 10:48:12PM -0500, Jason Pleau wrote:
>> I plan to maintain this package in collab-maint on alioth
> 
> Any progress here? I'm interested in tryint that stuff out...
> 
> .
> 

I originally replied on IRC but I'll go into more details here :

There are a few missing packages in Debian before I can start working on
polybar itself:

When we look at the lib/ folder there are these libraries that aren't in
the archive:

i3ipcpp (github.com/jaagr/i3ipcpp, forked from drmgc/i3pcpp)
  - auss (github.com/jaagr/auss, forked from drmgc/auss)
  - jsoncpp (seems to be in Debian as src:libjsoncpp)
xpp (github.com/jaagr/xpp, forked from jotrk/xpp)


I don't mind doing the work of packaging these 3 libraries, however I
would package the forks, unless someone wants to step in and try to get
the forks merged into their original upstreams.

There are also two headers included
  - concurrentqueue/include/moodycamel/blockingconcurrentqueue.h
  - concurrentqueue/include/moodycamel/concurrentqueue.h

These headers seem to be included in two other projects (radmap and
salmon). Is it ok to also include these directly with polybar ?

I'll try to get started on this soon, it's been a year since I filed the
ITP and I am still using polybar from a manual build / install..

Of course, I would welcome any help with this.

Thanks !


-- 
Jason Pleau

Bug#850741: python-docker 2.x in Sid

[Jason Pleau]

Hi Thomas

On 11/01/2017 08:51 PM, Thomas Goirand wrote:
> Hi,
> 
> Could you please upload the 2.x version of python-docker into Sid? I've
> uploaded python-zunclient that needs it.
> 


It's been a while, I think what we (still) have to do is get the
rdepends updated to work with the latest python-docker. I don't think
anyone looked at that yet.

We welcome help of course, I will take some time this weekend to see
what exactly needs to be done and start doing some work.

(sorry for the delays on this (I honestly just forgot to follow up..)

Thanks !


-- 
Jason Pleau

Bug#872794: i3blocks: [cpu_usage] Handle mpstat output change

[Jason Pleau]

Hi again.

On 08/21/2017 06:41 AM, Alok Singh wrote:
> Package: i3blocks
> Version: 1.4-3+b1
> Severity: normal
> 
> Dear Maintainer,
> 
> Not sure when this happened but the regexp in cpu_usage cannot parse the
> output of mpstat anymore. Patch attached.
> 

I investigated, and this is strange, I cannot reproduce your exact issue.

Out of curiosity, did you edit the cpu_usage script by yourself? The
reason I'm asking is that the change you're proposing is already in
unstable, see here:

http://sources.debian.net/src/i3blocks/1.4-3/scripts/cpu_usage/#L34

I will however add a patch that was recently added as a PR upstream (the
script does not work with newer kernel/mpstat):
https://github.com/vivien/i3blocks/pull/252




-- 
Jason Pleau

Bug#872794: i3blocks: [cpu_usage] Handle mpstat output change

[Jason Pleau]

Hi

2017-08-21 6:41 GMT-04:00 Alok Singh :

> Package: i3blocks
> Version: 1.4-3+b1
> Severity: normal
>
> Dear Maintainer,
>
> Not sure when this happened but the regexp in cpu_usage cannot parse the
> output of mpstat anymore. Patch attached.
>
>
> --
> Alok
>

Thanks for the report / patch. Looks like
https://github.com/vivien/i3blocks/issues/251 is also an issue related to
this. I'll "patch" your patch with this commit as well. Will get around to
it later this week

Cheers

Jason

Bug#867469: lua-cliargs: Please clean git tags for git getorig

[Jason Pleau]

Hi.

On 07/06/2017 01:58 PM, Jochen Sprickerhof wrote:
> Source: lua-cliargs
> Severity: minor
> 
> Dear Maintainer,
> 
> Could you please clean your git tags such that git deborig works?
> 
> $ git deborig
> tags upstream/3.0-1, v3.0-1 all exist in this repository
> tell me which one you want to make an orig.tar from: git deborig TAG
> 
> I assume v3.0-1 is the correct one (although it shouldn't have a hyphen in 
> there).

Here the upstream version is really 3.0-1 (a few lua packages I worked
on have this versionning; upstream add their luarocks revision as their
released version. I simply decided to go along and pick it up.)

To answer your question, shouldn't the correct tag here be
upstream/3.0-1 ? This is the one that was created (and is used) by git-gbp.

I saw you've filled other bugs for other lua packages, I'll do the same
on those as well once we have picked the correct tags.

> 
> Thanks!
> 
> Jochen
> 

Thanks !

-- 
Jason Pleau

Bug#856179: ITP: polybar -- fast and easy-to-use status bar

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: polybar
  Version : 3.0.4
  Upstream Author : Michael Carlberg <c...@rlberg.se>
* URL : https://github.com/jaagr/polybar/
* License : MIT/Expat
  Programming Lang: C++
  Description : fast and easy-to-use status bar

The main purpose of Polybar is to help users create awesome status bars. It has
built-in functionality to display information about the most commonly used
services, such as:

 * Systray icons
 * Window title
 * Playback controls and status display for MPD using libmpdclient
 * ALSA volume controls
 * Workspace and desktop panel for bspwm and i3
 * Workspace module for EWMH compliant window managers
 * Keyboard layout and indicator status
 * CPU and memory load indicator
 * Battery display
 * Network connection details
 * Backlight level
 * Date and time label
 * Time-based shell script execution
 * Command output tailing
 * User-defined menu tree
 * Inter-process messaging

Users can also develop their own modules and/or integrate modules created by
others.

It can be used a replacement for status bars / panels used in different window
managers and desktop environments.

I plan to maintain this package in collab-maint on alioth

Bug#847712: ITP: docker-pycreds -- Python bindings for the docker credentials store API

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: docker-pycreds
  Version : 0.2.1
  Upstream Author : Joffrey F <joff...@docker.com>
* URL : https://github.com/shin-/dockerpy-creds
* License : Apache-2.0
  Programming Lang: Python
  Description : Python bindings for the docker credentials store API

This package provides a python interface to docker-credential-helpers (a helper
to use the docker credentials store)

It is a new dependency for latest upstream version of python-docker.

Bug#846309: ITP: golang-github-docker-docker-credential-helpers -- Use native stores to safeguard Docker credentials

[Jason Pleau]

Hi

2016-11-29 23:00 GMT-05:00 Potter, Tim (HPE Linux Support)
:
> X-Debbugs-CC: debian-de...@lists.debian.org, 
> pkg-go-maintain...@lists.alioth.debian.org
> Package: wnpp
> Severity: wishlist
> Owner: Tim Potter 
>
> * Package name: golang-github-docker-docker-credential-helpers
>   Version : 0.3.0+git20160906.22.f72c04f-1
>   Upstream Author : David Calavera
> * URL : https://github.com/docker/docker-credential-helpers
> * License : Expat
>   Programming Lang: Go
>   Description : Use native stores to safeguard Docker credentials
>
> The docker-credential-helpers package is a suite of programs to use
>  the built-in OS native stores to keep Docker credentials safe.
> .
> It consists of a Go client and and an executable for manipulating the
> credential store.


See #843454 (https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=843454)

We have started work on the package in collab-maint, feel free to take
it over and move it to pkg-go, we will be maintaining the python
bindings, hence why we needed docker-credentials-helper

Bug#843454: ITP: docker-credential-helpers -- native store to keep Docker credentials, safe

[Jason Pleau]

Hi Felipe,

2016-11-16 17:40 GMT-05:00 Felipe Sateler <fsate...@debian.org>:
> Hi Jason
>
> On Sun, 6 Nov 2016 13:16:58 -0500 Jason Pleau <ja...@jpleau.ca> wrote:
>> Package: wnpp
>> Severity: wishlist
>> Owner: Jason Pleau <ja...@jpleau.ca>
>>
>> * Package name: docker-credential-helpers
>>   Version : 0.3.0
>>   Upstream Author : David Calavera <david.calav...@gmail.com>
>> * URL : https://github.com/docker/docker-credential-helpers/
>> * License : MIT
>>   Programming Lang: Go / C
>>   Description : native store to keep Docker credentials safe
>>
>> docker-credential-helpers provides a Linux executable to store Docker
>> credentials. (It also has one for OSX and Windows, but those won't be
>> built nor used in Debian).
>>
>> It is a dependency of an upcoming python package
>> "dockerpy-creds", which is needed to update python-docker to its latest
>> upstream version.
>
>
> Have you done some work yet on this? If so, could you push it to
> collab-maint so I can take a look?

Not yet, I scheduled some time saturday to work on it and hopefully
finalize it, I will keep you updated :)

Thanks !

>
> Saludos
>

Bug#843454: ITP: docker-credential-helpers -- native store to keep Docker credentials, safe

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: docker-credential-helpers
  Version : 0.3.0
  Upstream Author : David Calavera <david.calav...@gmail.com>
* URL : https://github.com/docker/docker-credential-helpers/
* License : MIT
  Programming Lang: Go / C
  Description : native store to keep Docker credentials safe

docker-credential-helpers provides a Linux executable to store Docker
credentials. (It also has one for OSX and Windows, but those won't be
built nor used in Debian).

It is a dependency of an upcoming python package
"dockerpy-creds", which is needed to update python-docker to its latest
upstream version.

-- 
Jason Pleau



signature.asc
Description: OpenPGP digital signature

Bug#835379: ITP: ruby-neovim -- ruby bindings for neovim

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: ruby-neovim
  Version : 0.3.0
  Upstream Author : Alex Genco <alexge...@gmail.com>
* URL : https://github.com/alexgenco/neovim-ruby
* License : MIT
  Programming Lang: Ruby
  Description : ruby bindings for neovim

Ruby library to communicate with neovim remotely

It is also a plugin host for neovim, allowing users to run and develop plugins
for neovim written entirely in Ruby.

It also acts as a compatibility layer for Ruby plugins written for vim. The
:ruby, :rubyfile, and :rubydo commands are intended to behave the same as they
did in vim

This package is required to enable ruby support in neovim, as well as enabling
the ruby tests for neovim.

I plan to maintain this package pkg-ruby

Bug#834461: lua-luv: FTBFS (32-bit): "basic 64bit conversions" test fails

[Jason Pleau]

Hi Aaron

On 08/15/2016 06:41 PM, Aaron M. Ucko wrote:
> Source: lua-luv
> Version: 1.9.1-0-1
> Severity: important
> Justification: fails to build from source
> 
> Builds of lua-luv for 32-bit platforms such as i386 have been failing:
> 
> ./tests/test-conversions.lua:3: assertion failed!
> stack traceback:
>   [C]: in function 'assert'
>   ./tests/test-conversions.lua:3: in function 'fn'
>   ./lib/tap.lua:53: in function <./lib/tap.lua:42>
>   [C]: in function 'xpcall'
>   ./lib/tap.lua:42: in function 'run'
>   ./lib/tap.lua:136: in function 'tap'
>   tests/run.lua:35: in main chunk
>   [C]: ?
>   not ok 2 conversions - basic 64bit conversions
> 
> Could you please take a look?

Thanks for reporting those

This one seems to be some weird behavior with lua5.1 on i386. I made
this test case:

http://paste.debian.net/789961/

The int conversions seems to be different on this architecture with
lua5.1. I'm investigating this and will followup as soon as I have news.

(For the other bugs you've reported #834462 and #834461 I'm waiting
access to porter machines to test my fixes)


> 
> Thanks!
> 

Cheers

-- 
Jason Pleau

Bug#832435: ITP: lua-nvim -- lua client for neovim

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: lua-nvim
  Version : 0.0.1-25
  Upstream Author : Neovim contributors
* URL : https://github.com/neovim/lua-client/
* License : Apache-2.0
  Programming Lang: Lua, C
  Description : lua client for neovim

Lua library to communicate with Neovim through its msgpack-rpc API

It will eventually be able to act as a Lua plugin host, enabling the use and
development of Lua plugins for Neovim.



This package is required to run the tests for neovim

It will be maintained in pkg-lua

Bug#832337: ITP: lua-inifile -- simple, complete ini parser for lua

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: lua-inifile
  Version : 1.0-2
  Upstream Author : Bart van Strien <bart@gmail.com>
* URL : https://github.com/bartbes/inifile
* License : Simplified BSD
  Programming Lang: Lua
  Description : simple, complete ini parser for lua

inifile is a simple, complete ini parser for lua that intends to preserve
as much information as possible, like the order of the file and the stored
comments. It is also out-of-the-box compatible with LÖVE.

This package is a dependency of lua-cliargs

It will be maintained in pkg-lua

Bug#832314: lua-yaml: please add lua5.3 support

[Jason Pleau]

Package: lua-yaml
Version: 6.0-2
Severity: wishlist

Dear Maintainer,

Please add lua5.3 support to lua-yaml. Otherwise other packages that depend on
lua-yaml cannot be packaged for lua5.3!

I think adding a symlink like it was done to add lua5.2 would be enough

Thanks !



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

Kernel: Linux 4.6.4 (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages lua-yaml depends on:
ii  libc6  2.23-2
ii  libyaml-0-20.1.6-3
ii  libyaml-dev0.1.6-3
ii  multiarch-support  2.23-2

lua-yaml recommends no packages.

lua-yaml suggests no packages.

Bug#824944: Update on #824944

[Jason Pleau]

Hey,

On 07/04/2016 06:43 PM, Julian Wollrath wrote:
> Hi,
> 
> thanks for your interest. I modified it myself to incorporate (and
> hopefully support) the flags of discount 2.1.8, therefore the version
> change, since upstream seems inactive. I am in contact with Enrico
> Tassi, who sponsored previous uploads to get it uploaded.
> 

Awesome, thanks for the swift reply and explanation !

> Cheers,
> Julian
> 

Cheers

-- 
Jason Pleau

Bug#824944: Update on #824944

[Jason Pleau]

Hi,

I saw you seem to have fixed the bug in git last week [1]

Is the new version going to be uploaded to unstable soonish ? I also
noticed the version string in debian/changelog didn't really match
anything that upstream released. (I can't find 2.1.8 anywhere)

I'd be glad to help if you need. (I have a package that has a build-dep
that depends on lua-discount, so I got notified)

1:
https://anonscm.debian.org/cgit/pkg-lua/lua-discount.git/commit/?id=25d62151f8ed866ee938eb6e8191771b22e834ca

Thanks !

-- 
Jason Pleau

Bug#829039: ITP: lua-luv -- libuv bindings for lua

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: lua-luv
  Version : 1.9.0-3
  Upstream Author : The Luvit Authors
* URL : https://github.com/luvit/luv
* License : Apache
  Programming Lang: C, Lua
  Description : libuv bindings for lua

libuv is a multi-platform support library with a focus on asynchronous I/O.

This package provides libuv bindings for the Lua programming language.


lua-luv is a dependency for lua-nvim (not yet packaged nor ITP'd), which in
turn is a dependency to run the tests for neovim, and in the future will likely
provide Lua bindings for neovim plugins.

I will maintain lua-luv in the pkg-lua team.

Bug#796497: ruby-ronn: depends on obsolete ruby-hpricot

[Jason Pleau]

Hi

On Tue, 17 Nov 2015 21:32:08 -0200 Antonio Terceiro
<terce...@debian.org> wrote:
> On Sun, Nov 15, 2015 at 01:37:23PM +0100, Sebastian Boehm wrote:
> > Just for the record: ronn's upstream seems to be dead. The latest
> > release is more than five years old and there haven't been any
> > upstream commits for over two years.
> > 
> > While ronn is a useful tool, asciidoc and asciidoctor are viable
> > alternatives and are actively maintained as well as available in
> > stretch and jessie.
> 
> I had a quick look at ronn and it looks like a lost case. it seems to convert
> the markdown input to HTML first, and only then from HTML to roff, and the
> dependency on hpricot seems very entrenched.
> 
> We currently have 24 packages build-depending on ruby-ronn:
> 
> $ grep-dctrl -F Build-Depends -n -s Package ruby-ronn Sources | sort -u

> i3blocks

I maintain the above package, and will contact upstream if another
format can be used instead of ronn so that i3blocks can be built on
stretch without issues.


Sorry for not noticing this bug earlier

> 
> -- 
> Antonio Terceiro <terce...@debian.org>

Thanks

-- 
Jason Pleau

Bug#822210: sdl2-config.cmake: extra leading / trailing whitespace

[Jason Pleau]

Hi

On 05/19/2016 07:10 AM, Manuel A. Fernandez Montecelo wrote:
> Hi!
> 
> 2016-05-01 15:42 Manuel A. Fernandez Montecelo:
>>
>> If it needs this change, I think that my solution is not very robust
>> and that the assumptions made don't work and can fail in other cases.
> 
> So I've been fighting with this for a while this morning and couldn't
> get it to work as originally intended.  In the end, I uploaded a new
> version of the package with the patch stripping only the SDL2_LIBRARIES
> variable.
> 
> If other vars contain whitespace in the future it will not work, but
> that's fundamentally a problem with upstream...  I submitted the patch
> for their consideration in any case.
> 
> It would be great if you can verify that this works fine in your system,
> but I am quite confident that it does so I published the new release to
> the archive already -- lest it slips through the cracks and it's delayed
> for a few more weeks.

Seems to work here, thanks for the fix and upload!

> 
> 
> Cheers.

-- 
Jason Pleau

Bug#822210: sdl2-config.cmake: extra leading / trailing whitespace

[Jason Pleau]

Hi (removed the SDL maintainers/developers from this email, don't want
to spam..)

On 04/29/2016 06:44 AM, Manuel A. Fernandez Montecelo wrote:

... [snip] ...

> 
> At a first glance, it looks very odd for cmake to be so fussy about
> whitespace, but I guess that they have a good reason.
> 
> I've been looking into it and a more definitive/robust fix could be to
> trim the strings coming from configure, so the same problem is avoided
> in other systems or in future changes to other variables.  (Patch
> attached).
> 
> Since "string(STRIP input output)" does not return the stripped
> string, instead of using the string() function in the same spot where
> the original variable is used, one has to use an "output variable" and
> then use the variable as substitute in the original command.
> 
> So the result looks a bit ugly and duplicates the number of lines, and
> everything is a bit cumbersome, but I think that it works.
> 
> Jason, if you want to test it to see if it works fine, then I can
> submit both to upstream's bug report for them to decide.

It looks like ${prefix} is not set when defining ${stripped_prefix}. So
for example SDL2_INCLUDE_DIRS is set to /include/SDL2.

I got it to work with the following:

# sdl2 cmake project-config input for ./configure scripts

string(STRIP "@prefix@" prefix)
string(STRIP "@exec_prefix@" stripped_exec_prefix)
string(STRIP "@libdir@" stripped_libdir)
string(STRIP "@includedir@" stripped_includedir)
string(STRIP "-L${SDL2_LIBDIR} @SDL_RLD_FLAGS@ @SDL_LIBS@"
stripped_SDL2_LIBRARIES)

set(exec_prefix "${stripped_exec_prefix}")
set(libdir "${stripped_libdir}")
set(SDL2_PREFIX "${stripped_prefix}")
set(SDL2_EXEC_PREFIX "${stripped_prefix}")
set(SDL2_LIBDIR "${stripped_libdir}")
set(SDL2_INCLUDE_DIRS "${stripped_includedir}/SDL2")
set(SDL2_LIBRARIES "${stripped_SDL2_LIBRARIES}")


(patch format attached)


> 
> (If anybody wants to use the attached patch as a basis for better
> patches, feel free to do it.  Also anybody feel free to submit the
> patches upstream, but better if tested first).
> 
> 
> Cheers.
> 

Thanks !

-- 
Jason Pleau
--- a/sdl2-config.cmake.in
+++ b/sdl2-config.cmake.in
@@ -1,10 +1,15 @@
 # sdl2 cmake project-config input for ./configure scripts
 
-set(prefix "@prefix@") 
-set(exec_prefix "@exec_prefix@")
-set(libdir "@libdir@")
-set(SDL2_PREFIX "@prefix@")
-set(SDL2_EXEC_PREFIX "@prefix@")
-set(SDL2_LIBDIR "@libdir@")
-set(SDL2_INCLUDE_DIRS "@includedir@/SDL2")
-set(SDL2_LIBRARIES "-L${SDL2_LIBDIR} @SDL_RLD_FLAGS@ @SDL_LIBS@")
+string(STRIP "@prefix@" prefix)
+string(STRIP "@exec_prefix@" stripped_exec_prefix)
+string(STRIP "@libdir@" stripped_libdir)
+string(STRIP "@includedir@" stripped_includedir)
+string(STRIP "-L${SDL2_LIBDIR} @SDL_RLD_FLAGS@ @SDL_LIBS@" stripped_SDL2_LIBRARIES)
+
+set(exec_prefix "${stripped_exec_prefix}")
+set(libdir "${stripped_libdir}")
+set(SDL2_PREFIX "${stripped_prefix}")
+set(SDL2_EXEC_PREFIX "${stripped_prefix}")
+set(SDL2_LIBDIR "${stripped_libdir}")
+set(SDL2_INCLUDE_DIRS "${stripped_includedir}/SDL2")
+set(SDL2_LIBRARIES "${stripped_SDL2_LIBRARIES}")

Bug#822210: sdl2-config.cmake: extra leading / trailing whitespace

[Jason Pleau]

Hi Gianfranco,

On Fri, 22 Apr 2016 14:23:53 +0200 Gianfranco Costamagna
<locutusofb...@debian.org> wrote:
> Hi,
> > set(SDL2_LIBRARIES "-L${SDL2_LIBDIR}  -lSDL2 ")
> 
> I know where that space comes from:
> 
> quoting configure.in
> 
> SDL_LIBS="-lSDL2 $BASE_LDFLAGS"
> 
> 
> changing to 
> SDL_LIBS="-lSDL2$BASE_LDFLAGS"
> 
> fixes the issue, because in our case BASE_LDFLAGS is undefined.
> 
> I'm not sure why the cmake build is not even creating it, and I'm not sure how
> to best fix the issue.
> 
> I think a patch would be welcome, so the maintainers can apply it if needed :)

I attached a patch, would something like this be an acceptable solution
? If yes I think we should forward upstream

> 
> thanks for the bug report,
> 
> Gianfranco
> 

-- 
Jason Pleau
>From bdab9ab551bd388f3a985909a5534535899048cc Mon Sep 17 00:00:00 2001
From: Jason Pleau <ja...@jpleau.ca>
Date: Sun, 24 Apr 2016 13:13:35 -0400
Subject: [PATCH] patch to remove trailing whitespace in sdl2-config.cmake

---
 debian/changelog   |  5 +++-
 .../patches/remove_trailing_whitespace_cmake.patch | 27 ++
 debian/patches/series  |  1 +
 3 files changed, 32 insertions(+), 1 deletion(-)
 create mode 100644 debian/patches/remove_trailing_whitespace_cmake.patch

diff --git a/debian/changelog b/debian/changelog
index 3c85867..59ff8a5 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -22,7 +22,10 @@ libsdl2 (2.0.4+dfsg2-1) UNRELEASED; urgency=medium
   this repack can be removed in the next upstream tarball import.
   (Closes: #814276).
 
- -- Manuel A. Fernandez Montecelo <m...@debian.org>  Sun, 21 Feb 2016 19:08:44 +
+  [ Jason Pleau ]
+  * patch to remove trailing whitespace in sdl2-config.cmake (Closes: #822210)
+
+ -- Jason Pleau <ja...@jpleau.ca>  Sun, 24 Apr 2016 13:08:58 -0400
 
 libsdl2 (2.0.4+dfsg1-2) unstable; urgency=low
 
diff --git a/debian/patches/remove_trailing_whitespace_cmake.patch b/debian/patches/remove_trailing_whitespace_cmake.patch
new file mode 100644
index 000..c9d5f0a
--- /dev/null
+++ b/debian/patches/remove_trailing_whitespace_cmake.patch
@@ -0,0 +1,27 @@
+Description: remove the trailing whitespace in sdl2-config.cmake
+ CMake complains that there is trailing whitespace for the SDL2_LIBARIES
+ directive. This patch removes the space before $BASE_LDFLAGS in configure.in
+ which results in no extra whitespace in sdl2-config.cmake.
+Author: Jason Pleau <ja...@jpleau.ca>
+Bug-Debian: https://bugs.debian.org/822210
+
+--- a/configure.in
 b/configure.in
+@@ -97,7 +97,7 @@
+ 
+ if test x$have_no_cygwin = xyes; then
+ BASE_CFLAGS="-mno-cygwin"
+-BASE_LDFLAGS="-mno-cygwin"
++BASE_LDFLAGS=" -mno-cygwin"
+ fi
+ BASE_CFLAGS="$BASE_CFLAGS -I/usr/include/mingw"
+ ;;
+@@ -123,7 +123,7 @@
+ #fi
+ #done
+ SDL_CFLAGS="$BASE_CFLAGS"
+-SDL_LIBS="-lSDL2 $BASE_LDFLAGS"
++SDL_LIBS="-lSDL2$BASE_LDFLAGS"
+ CPPFLAGS="$CPPFLAGS $EXTRA_CFLAGS"
+ CFLAGS="$CFLAGS $EXTRA_CFLAGS"
+ LDFLAGS="$LDFLAGS $EXTRA_LDFLAGS"
diff --git a/debian/patches/series b/debian/patches/series
index 642e32f..8f1b1e9 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
+remove_trailing_whitespace_cmake.patch
 SDL2_dont_propagate_lpthread.diff
 replace-relicenced-SDL_qsort.patch
-- 
2.8.1

Bug#822210: sdl2-config.cmake: extra leading / trailing whitespace

[Jason Pleau]

Package: libsdl2-dev
Version: 2.0.4+dfsg1-2+b1
Severity: normal

Dear Maintainer,

When trying to use SDL2 in a small project with CMake, I encountered the
following error:

CMake Error at CMakeLists.txt:18 (add_executable):
  Target "run" links to item "-L/usr/lib/x86_64-linux-gnu -lSDL2 " which has
  leading or trailing whitespace.  This is now an error according to policy
  CMP0004.


This is the relevant part of my own CMakeLists.txt (line 18):

--

find_package(SDL2 REQUIRED)

--


The offending line is in /usr/lib/x86_64-linux-gnu/cmake/SDL2/sdl2-config.cmake
(could be different depending on the arch used)

Line #10:

--

set(SDL2_LIBRARIES "-L${SDL2_LIBDIR}  -lSDL2 ")

--

Removing the trailing whitespace after -lSDL2 fixes the issue for me. (There's
also an extra leading whitespace before -lSDL2)

I am not certain how to fix this properly in the sdl2-config.cmake.in file,
otherwise I would have provided a patch.

For the info, the version of CMake I am running: 3.5.1-1

Thanks !



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

Kernel: Linux 4.5.2-jpleau (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages libsdl2-dev depends on:
ii  libasound2-dev 1.1.0-1
ii  libdbus-1-dev  1.10.8-1
ii  libegl1-mesa-dev   11.1.3-1
ii  libgl1-mesa-dev11.1.3-1
ii  libgles2-mesa-dev  11.1.3-1
ii  libglu1-mesa-dev   9.0.0-2.1
ii  libpulse-dev   8.0-2+b2
ii  libsdl2-2.0-0  2.0.4+dfsg1-2+b1
ii  libsndio-dev   1.1.0-2
ii  libudev-dev229-4
ii  libwayland-dev 1.10.0-2
ii  libx11-dev 2:1.6.3-1
ii  libxcursor-dev 1:1.1.14-1+b1
ii  libxext-dev2:1.3.3-1
ii  libxi-dev  2:1.7.6-1
ii  libxinerama-dev2:1.1.3-1+b1
ii  libxkbcommon-dev   0.5.0-1
ii  libxrandr-dev  2:1.5.0-1
ii  libxss-dev 1:1.2.2-1
ii  libxt-dev  1:1.1.5-1
ii  libxv-dev  2:1.0.10-1+b1
ii  libxxf86vm-dev 1:1.1.4-1

libsdl2-dev recommends no packages.

libsdl2-dev suggests no packages.

-- no debconf information

Bug#818112: ITP: neovim-qt -- neovim client library and GUI

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: neovim-qt
  Version : 0.1.1
  Upstream Author : Rui Abreu Ferreira <raf...@gmx.com>
* URL : https://github.com/equalsraf/neovim-qt
* License : ISC
  Programming Lang: C++
  Description : neovim client library and GUI

It provides Qt5 bindings (libneovim-qt) to help developing neovim clients and
plugins.

It also provides a GUI for neovim, using the above bindings.

This package will be maintained in the pkg-vim team

Bug#816566: lua-term: Add support for Lua5.1 and Lua5.3

[Jason Pleau]

Package: lua-term
Version: 0.03.20140426.g1ba9a8e-1
Severity: wishlist
Tags: patch

Dear Maintainer,

To be able to run the tests for neovim (tests are written in Lua, they are ran
by 'busted' which a package currently being worked on), we need lua-term to be
able to run on Lua5.1

I included a patch that adds support for lua5.1 and lua5.3 (I tested the small
example that upstream provides, it runs correctly)

Cheers



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

Kernel: Linux 4.4.3-jpleau (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages lua-term depends on:
ii  libc6  2.21-9

lua-term recommends no packages.

lua-term suggests no packages.
>From f86c3b8fc74202cb06549f3be14adddc86320295 Mon Sep 17 00:00:00 2001
From: Jason Pleau <ja...@jpleau.ca>
Date: Wed, 2 Mar 2016 20:26:19 -0500
Subject: [PATCH] Add support for Lua5.1 and Lua5.3

---
 debian/dh-lua.conf| 24 
 debian/lua5.1.dh-lua.conf | 23 +++
 debian/lua5.2.dh-lua.conf |  1 +
 debian/lua5.3.dh-lua.conf |  1 +
 4 files changed, 25 insertions(+), 24 deletions(-)
 delete mode 100644 debian/dh-lua.conf
 create mode 100644 debian/lua5.1.dh-lua.conf
 create mode 12 debian/lua5.2.dh-lua.conf
 create mode 12 debian/lua5.3.dh-lua.conf

diff --git a/debian/dh-lua.conf b/debian/dh-lua.conf
deleted file mode 100644
index 7864169..000
--- a/debian/dh-lua.conf
+++ /dev/null
@@ -1,24 +0,0 @@
-### mandatory fields
-LUA_VERSION=5.2
-PKG_NAME=term
- 
-### things relative to the C library part
-CLIB_CFLAGS=
-CLIB_LDFLAGS=
-CLIB_LDFLAGS_STATIC=
-CLIB_OBJS=core.lo
-LUA_MODNAME_CPART=term.core
- 
-### things relative to the lua library part
-LUA_HEADER=
-LUA_SOURCES=$(wildcard term/*.lua)
-LUA_SOURCES_MANGLER=
-LUA_MODNAME=term
-LUA_TEST=
- 
-### this part is relative to pkg-config
-PKG_VERSION=
-PKG_LIBS_PRIVATE=
-PKG_URL=
-PKG_REQUIRES=
-PKG_CONFLICTS=
diff --git a/debian/lua5.1.dh-lua.conf b/debian/lua5.1.dh-lua.conf
new file mode 100644
index 000..0ade434
--- /dev/null
+++ b/debian/lua5.1.dh-lua.conf
@@ -0,0 +1,23 @@
+### mandatory fields
+PKG_NAME=term
+ 
+### things relative to the C library part
+CLIB_CFLAGS=
+CLIB_LDFLAGS=
+CLIB_LDFLAGS_STATIC=
+CLIB_OBJS=core.lo
+LUA_MODNAME_CPART=term.core
+ 
+### things relative to the lua library part
+LUA_HEADER=
+LUA_SOURCES=$(wildcard term/*.lua)
+LUA_SOURCES_MANGLER=
+LUA_MODNAME=term
+LUA_TEST=
+ 
+### this part is relative to pkg-config
+PKG_VERSION=
+PKG_LIBS_PRIVATE=
+PKG_URL=
+PKG_REQUIRES=
+PKG_CONFLICTS=
diff --git a/debian/lua5.2.dh-lua.conf b/debian/lua5.2.dh-lua.conf
new file mode 12
index 000..ba875b6
--- /dev/null
+++ b/debian/lua5.2.dh-lua.conf
@@ -0,0 +1 @@
+lua5.1.dh-lua.conf
\ No newline at end of file
diff --git a/debian/lua5.3.dh-lua.conf b/debian/lua5.3.dh-lua.conf
new file mode 12
index 000..ba875b6
--- /dev/null
+++ b/debian/lua5.3.dh-lua.conf
@@ -0,0 +1 @@
+lua5.1.dh-lua.conf
\ No newline at end of file
-- 
2.7.0

Bug#815489: ITP: lua-system -- platform independent system call library for Lua

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau <ja...@jpleau.ca>

* Package name: lua-system
  Version : 0.1.0
  Upstream Author : Oscar Lim <o...@ucla.edu>
* URL : https://github.com/o-lim/luasystem
* License : Expat
  Programming Lang: C, Lua
  Description : platform independent system call library for Lua


luasystem is a platform independent system call library for Lua.

It supports Lua >= 5.1 and luajit >= 2.0.0.

Currently the following functions are supported:

 * gettime
 * sleep

This package is a dependency of busted (#783044), which is required to run the
tests for neovim.

The package will be maintained in pkg-lua.

Bug#812531: qtcreator: ClangCodeModel plugin not working after update to 3.6.0

[Jason Pleau]

Package: qtcreator
Version: 3.6.0-1
Severity: normal

Dear Maintainer,

Since updating QtCreator to 3.6.0 this week, I noticed that the ClangCodeModel
plugin stopped working (does not give any completions)

I tried with a clean configuration with no luck

Downgrading to 3.5.1+dfsg-2 makes it work again.

Hopefully I'm not alone with this issue so others can reproduce!

Thanks for maintaining QtCreator!



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

Kernel: Linux 4.4.0-jpleau (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages qtcreator depends on:
ii  libbotan-1.10-1   1.10.10-6+b1
ii  libc6 2.21-6
ii  libclang1-3.6 1:3.6.2-3
ii  libgcc1   1:5.3.1-7
ii  libqbscore1   1.4.4+dfsg-1
ii  libqbsqtprofilesetup1 1.4.4+dfsg-1
ii  libqt5concurrent5 5.5.1+dfsg-12
ii  libqt5core5a [qtbase-abi-5-5-1]   5.5.1+dfsg-12
ii  libqt5designer5   5.5.1-3
ii  libqt5designercomponents5 5.5.1-3
ii  libqt5gui55.5.1+dfsg-12
ii  libqt5help5   5.5.1-3
ii  libqt5network55.5.1+dfsg-12
ii  libqt5printsupport5   5.5.1+dfsg-12
ii  libqt5qml5 [qtdeclarative-abi-5-5-0]  5.5.1-3
ii  libqt5quick5  5.5.1-3
ii  libqt5quickwidgets5   5.5.1-3
ii  libqt5sql55.5.1+dfsg-12
ii  libqt5sql5-sqlite 5.5.1+dfsg-13
ii  libqt5webkit5 5.5.1+dfsg-2
ii  libqt5widgets55.5.1+dfsg-12
ii  libqt5xml55.5.1+dfsg-12
ii  libstdc++65.3.1-7
ii  qml-module-qtquick-controls   5.5.1-2
ii  qml-module-qtquick2   5.5.1-3
ii  qtchooser 52-gae5eeef-2
ii  qtcreator-data3.6.0-1

Versions of packages qtcreator recommends:
ii  gdb   7.10-1+b1
ii  gnome-terminal [x-terminal-emulator]  1:3.18.1-1+jpleau1
ii  make  4.1-4
pn  qt5-doc   
ii  qtbase5-dev-tools 5.5.1+dfsg-12
pn  qtcreator-doc 
ii  qtdeclarative5-dev-tools  5.5.1-3
ii  qttools5-dev-tools5.5.1-3
ii  qttranslations5-l10n  5.5.1-2
ii  qtxmlpatterns5-dev-tools  5.5.1-2
ii  xfce4-terminal [x-terminal-emulator]  0.6.3-2
ii  xterm [x-terminal-emulator]   322-1

Versions of packages qtcreator suggests:
ii  cmake  3.4.1-2
ii  g++4:5.3.1-1
ii  git1:2.7.0-1
pn  kdelibs5-data  
pn  subversion 

-- no debconf information

Bug#811034: Please add Jason Pleau as Debian Maintainer

[Jason Pleau]

Package: debian-maintainers
Severity: normal

Hi,

Please add my key (0xC7B0B792) to the Debian Maintainers keyring.

I have attached the jetring changeset.

Thanks !

-- 
Jason Pleau
Comment: Please add Jason Pleau <ja...@jpleau.ca> as a Debian Maintainer
Date: Thu, 14 Jan 2016 21:59:18 -0500
Action: import
Recommended-By: 
  Michael Stapelberg <stapelb...@debian.org>,
  James McCoy <james...@debian.org>
Agreement: 
  https://lists.debian.org/debian-newmaint/2016/01/msg5.html
Advocates: 
  https://lists.debian.org/debian-newmaint/2016/01/msg00010.html,
  https://lists.debian.org/debian-newmaint/2016/01/msg00014.html
Data: 
  -BEGIN PGP PUBLIC KEY BLOCK-
  Version: GnuPG v1
  
  mQINBFRavRsBEAC5eU+998UUYQO6C88Zl6TB1HA3OB2JNBxJRtKvOLoap5oKmsVn
  dzaGutjN2cq1awWzIMELSTneUaHFtKaYeFhmcgBU3FQszVvoskE3c+Nvq/Uyt6Xp
  c+MpcKjsOGRLTeWnv0Kw5i5OQD+gy7NcwwuGUMmHFZHHSPTZ5wXdyslqJ0DaZP4D
  Jqkbxi1Cx05/gIk3LdGCdCLiPsA5TjUDjCMwdSZtXjkp1TrVIS4X1yEM+VX0t9Is
  PphPFCR6ZvvfRiShdF2V96BMeYZItfePDBSi4wZcdqeGr3sCpVqoh52yNiwBDTMu
  D36yUJyW2AUn7EVDy4C560MVORoaL3S64Q28KP4sNv32QpbkgToLIJKBDFo077ep
  sMmUuZuymFqXSLO44f6VQtnjPY9d4bojyUuQNICdLMToQmV9ONI7uK/oExYBAN8N
  mS9RlfzQvaEEuobuYU06F1Bc1R6LLyZPVXV9MJlihX1odxpKlycWd3j6SUfPpenf
  hhxusFJIOwqODPyD/AB5qdoXnf7ofOEGpMvNgRg5fQ/YbNX997d8jC82pedjklKC
  M8qiD1iUZ2BKl7L3Jr8ZNzfs3W+8a4zK1jwFw3Z6cFMPvLdowvxMTsTzlrX1yzHK
  0UjSvYn3dXWaf248nJr6B2J/dWM4EI8HWZupplBtC3HBGaPM9LEU4dI5vQARAQAB
  tB1KYXNvbiBQbGVhdSA8amFzb25AanBsZWF1LmNhPokCPQQTAQgAJwUCVFq9GwIb
  AwUJEswDAAULCQgHAwUVCgkICwUWAgMBAAIeAQIXgAAKCRAmEWSKx7C3kpMXD/0e
  oi0IclDBAifyojw55J4sE8jkxgg8p7+ubAKMec+tgud4nKnZCNiRPcprE+TZrG62
  jj/S9frs3JFJPzRd/ZWLfVW9oO5t1L0tu3POpy+t6Hfl/hq/YO60lx80vLKsCrsE
  TPg2Cgo3gkQbI95xBPWCYenFp+GREcWi+xQOYITwBVtxGVqNM+8NDLJWXoZI40uv
  M3MN5aqwRZGrMnEkwQBNWhCWE31HPdYR0AYnVK3aM7i4zIgBKKhNqE0lCgFDEjUJ
  az9faBMZ+XJQG4rGililTMLrtfur4VxaBkdamRtbj5jeAguBURaLvsPgPXD0uH9u
  G6JISa3h70w256OMM4z5rShwSo7J6Cp7gDtJgXL3inJM964AcCX6yrtSQkjK3lL+
  lY1+G+iMkyP4xZSmOg4uIh6ae/HP/kVxVCbzTClyqCqQQrNXxBQP48pGQ7wGuvGR
  PqDiZUnuoE2m/aqhkp8crIhNvf39X250PjYUEnA4/wNXYZ2lv1LBC8aRQT5wOfkM
  r11LESKRzm7u1CYcwnGjBWFOpnbUx0OK3b/HkROPItxCDqTOsM6Z5LoguzBqRg5U
  hK9WcEZlK89dMxHKCSQPB+kOWoMPqiqg0m/v7zZiAdt6OaErxuNYMJ4lR5842l1S
  j5AC/qb1UPAFzxW9IfuCnv2ncc1U8vNTdRNbn72tGYkCHAQQAQIABgUCVnxDYgAK
  CRDagoMOPMw6OleTD/kB0XAPJD2pd+pDkk8Y+UpY6bylBUUaFtcqE46eORhiriij
  ndaWlYu658JM1Hh4aF2L/IWSUMhgVhTvBIGmfQJYnyMR+b2HO6pnk4srrPX+Gg1E
  PxXndShtLfc6R7PedLhcNqZ9292IhV/X5i3cLX92LQKuzJrIgUXH9Ef4hh+0+ZT3
  tITG6A1eELOYWgjEdE5Rl8YTA2p1JYLCKU0Xia2ao3dDYI1mPjxyVwK/RDjxEmp0
  3U/0W0QGxIjHDuOSDkOEwsQY1OdvPbeSCnSPDtdO0kqFjwLEVgCt/dfqbMa606uC
  lElhWhFbUj9iRXn0rGQLJVc9cs9qObA6iPHmz9G612k9+0PistYT7dG2kNDsTrn+
  8HYQXTjiSCSwQIDlQTHv5mlwgw9rVU2N+teJZ1kD2vBc9NMMRAIDno5t0tvZUciX
  p2obLDZkVAvph/Nh/DdlIniy6/+buMaMbj+CbJleRBYq9pw3EXU4WGHieD3KYFey
  F7ADGFljetcaa/fGQm8ID06+oaqxUgPR6P2mDdtbANURkzq5aY0DMbJgcbUIHWIb
  OtEdAbCeQLRz6dwHA4AlyFbk595uOPwExNsrMH3iTo25MdHSSEEkbU3duzW55k+F
  brsj9Wsppd+nIgFF1Ks4SYbNu5ZEQneBxbD+XxURrIerpeR/PC62I/W+Y3+WNLkC
  DQRUWr0bARAAtCtvnTuOp7JbToP/0c0J/PXVrAzhxUyAmV3l4hmNzexe7S4nCIoo
  XB96tEmr2R7m5/iTYwMZgtAjo/D+3D/Jw7/AJ27ziDJ02JPQeY0gs5jM9LIQlquJ
  MJxABBSz6M0M8yiFnEcT37GJaxo6GCvYta1DGOzK0AEi8efm0SnM/GkeAN04IgJu
  E+oNia1Ei75o6+PuN4WuTskKhYi5wrn4AWwjdR/c4JBCjqFeBsJFf+9PP8j+URVU
  ytuZZtE3btWBtOnosrPyzECmKYvL8+2NYr9snP8tVMJt0Shls5IU20BeMxDv3Q0C
  8TUmaA4zGoDUBqaRyWolOB7vgzYwCyJsgxZrQQald4QyCg5sFvp7FNlRW2MhwaMo
  6+bbD28o2cwo5RwENUTa2ONdkEmCtqa3zLTKmMAJlOXsJolP3mDfTzjFfk8J1q/p
  DTGHD0H8SFNSx9wicNmB1A52DfQ/y6BtJHTW+gVEcdo7YpaHy1koWYmI/wFp4yzi
  fTvUcDjGoN0atznbtuyWWTZJgqBjsBRBdUQFIWZJK16Fgg3m0bgKWYctYx0pJn+8
  mgjLeeoWu4unooeStijXtQu9JHrysay85TEbEnyGcYdaKz3KeZhfB1T24xMOJGmh
  RG+ojFRbCjI6CJFtrkYcxSH8WczaTD5uq+YwPmb1OsU1KaFOZcvxyEcAEQEAAYkC
  JQQYAQgADwUCVFq9GwIbDAUJEswDAAAKCRAmEWSKx7C3kmyZEACqpWtzCxfiQ0ce
  gU9gRrG8XivenLk6XexOsnClBwa/oQOhO9JiRrC+F4HPh/694F8BQYteQNyCVLUP
  lNaMe+ywuxy0mI5nCkDIF8idumWBFxnRoWaBmV3tkArBsnSEKnykUU2HAt67W2zw
  nDy5i+QDcWAUPyoqca8in4aKE31rabuU3pwUfMGAjyTHO7kvYx0TM4Xyp4sBXAk2
  jBUWUNRYMNxA2sc9R9A9WbFyRB2XG+mBG6PaSrGy+AYXcopr0BXajhSjr26LFHp+
  QX6/4p29B6yshB2GKq5b7U0Io48so86CSzaejGoRF/T2khivvnJszg7wCfbkmSWJ
  ARQ72TMAqLMc7FX2/OQQXlnwCG+uGSXVM1z5Ctat0ZHs4wEwmbnQ53Nzut3lEnMs
  GPcxF9SUoP9ynpW+jdGX7AJQT7Xi4hnEVZbauMzW1DzKraC8VEIjQitxCtrlSkeE
  mowkFbA//+IbVesE2DP/+YGNVh4Y/V5suvGWsO9Sd+oPi4GWvam+0DFA0A5Br3O5
  pbt2VEKtDrwuS9AM48ORbXUUv/tHcaPaYaYxcL4yBhdt6LX/6tFcGGS73rJ0VqhJ
  MTIGIJSdOLtLiMuQ8OA2jAh2RTMC8QXlbVdHV7IIUpQW1j/93S/XXcrQx4Hxj4j7
  f9l5F+v/J4IqFt5+tx/MKLK/hMFXS7kCDQRUYWltARAAyUPsyXFMGzYMddVjOD1j
  mg6bKQ5Z1DyP1iexwbzdSy7eecs7Nbq9pzMco5ibyXGkOx1QpDcegSscGxllsCJY
  +wmhsTCKQKc8EhUNRjOYu5NweNznQRS/zgrQnRo9+yhyWlPH3T1JBtZFcMFv0QYa
  DtGOVvvt8ADMK2yu++DCx1auCj8VejGQk223V6IF0eH6rD+YnKrm05JgpipmhsL9
  57Lrk4PHSoEnnAg5PP0fIp/ZM/BPeLHg3N3KMlOpxz0cY8djatj/zY4sJfVj/Txh
  qyelVoVH9GmZ+eT9S3xJqoxSZY3hUgvCx7Ud/zLIYEEZppVh30cH+Ti504h4SXPi
  DLjjawiMSgdTAPkhSNU/mSt8A1lTtH3/ZgRacsoe0QLP4/

Bug#803994: rofi: segfaults on -show window (upstream#256)

[Jason Pleau]

Hi,

On 11/03/2015 08:57 PM, Guy Hughes wrote:
> Package: rofi
> Version: 0.15.10-1
> Severity: grave
> Tags: upstream
> Justification: renders package unusable
> 
> Dear Maintainer,
> 
> It may be of note that upstream release 0.15.10 has serious bugs:
> 
>* https://github.com/DaveDavenport/rofi/issues/256 (confirmed)
>* https://github.com/DaveDavenport/rofi/issues/258 (possible duplicate)
> 
> A complete upstream patch is not available.
> 
> 
>* What led up to the situation?
> 
>`rofi -show window` or `rofi -key-run alt+r` 
>
>* What was the outcome of this action?
> 
> (process:30936): GLib-CRITICAL **: g_strchug: assertion 'string != 
> NULL' failed
> (process:30936): GLib-CRITICAL **: g_strchomp: assertion 'string != 
> NULL' failed
> zsh: segmentation fault (core dumped)  rofi -show window
> 

Do you have any rofi settings in $HOME/.Xresources you can provide in
this bug report?

What desktop environment and/or window manager are you running rofi with ?

I cannot reproduce #256, but can easily reproduce #258 (with a high
enough 'padding' in arguments or in .Xresources)

It looks like a patch that attempts to fix #256 was commited:
https://github.com/DaveDavenport/rofi/commit/f9e7bc0775724a348e14c32437eeaffc255a6ffd

But I cannot test whether or not it fixes the problem since I can't
reproduce it =/

Are you able to test if the above commit fixes the issue for you ? I can
provide a temporary patched package if you cannot build rofi yourself.

As for #258 I have ran a git bisect and updated the bug upstream, I'm
sure it will help the developer fix it :)

> Thanks,
> 
> Guy Hughes
>

Thanks for bringing this up !

-- 
Jason Pleau

Bug#801517: lintian: broken link to Debian Packaging Policy for Vim

[Jason Pleau]

Hi

On 10/11/2015 11:48 AM, Jakub Wilk wrote:
> Package: lintian
> Version: 2.5.38
> Severity: minor
> 
> https://lintian.debian.org/tags/vim-addon-within-vim-runtime-path.html
> links to http://pkg-vim.alioth.debian.org/vim-policy.html/, but the
> pkg-vim.alioth.debian.org domain doesn't exist.
> 
> Dear vim maintainers, would it possible to make Debian Packaging Policy
> for Vim available online again? (Perhaps
> https://www.debian.org/doc/devel-manuals would be a safer place to host
> it than Alioth.)
> 

http://pkg-vim.alioth.debian.org/vim-policy.html/ works for me here

jason ~ ping pkg-vim.alioth.debian.org
PING pkg-vim.alioth.debian.org (5.153.231.21) 56(84) bytes of data.
64 bytes from moszumanska.debian.org (5.153.231.21): icmp_seq=1 ttl=51
time=102 ms
64 bytes from moszumanska.debian.org (5.153.231.21): icmp_seq=2 ttl=51
time=103 ms


-- 
Jason Pleau

Bug#794434: d-feet: segfault when trying to browse org.freedesktop.systemd1

[Jason Pleau]

Package: d-feet
Version: 0.3.9-1
Severity: normal

Dear Maintainer,

I installed d-feet to be able to browse the dbus API that systemd exposes.
Unfortunately, trying to browse org.freedesktop.systemd1 crashes d-feet with a
segmentation fault.

1. Launch d-feet
2. Double click on org.freedesktop.systemd1
3. It crashes

This is with version 0.3.9-1 from sid.

Here's the output I captured:

jason ~ d-feet
/usr/lib/python2.7/dist-packages/dfeet/uiloader.py:38: Warning: The property
GtkWidget:margin-left is deprecated and shouldn't be used anymore. It will be
removed in a future version.
  self.ui.add_from_file(self.ui_dir + '/' + f)
/usr/lib/python2.7/dist-packages/dfeet/uiloader.py:38: Warning: The property
GtkWidget:margin-right is deprecated and shouldn't be used anymore. It will be
removed in a future version.
  self.ui.add_from_file(self.ui_dir + '/' + f)
Gtk-Message: GtkDialog mapped without a transient parent. This is discouraged.
Exception RuntimeError: 'maximum recursion depth exceeded' in  ignored
Traceback (most recent call last):
  File /usr/lib/python2.7/dist-packages/dfeet/introspection.py, line 183, in
__dbus_node_introspect_cb
tree_iter = self.__treemodel.append(tree_iter, [%s % object_path,
node_obj])
  File /usr/lib/python2.7/dist-packages/gi/overrides/Gtk.py, line 1207, in
append
return self._do_insert(parent, -1, row)
  File /usr/lib/python2.7/dist-packages/gi/overrides/Gtk.py, line 1199, in
_do_insert
row, columns = self._convert_row(row)
  File /usr/lib/python2.7/dist-packages/gi/overrides/Gtk.py, line 857, in
_convert_row
result.append(self._convert_value(cur_col, value))
  File /usr/lib/python2.7/dist-packages/gi/overrides/Gtk.py, line 875, in
_convert_value
return GObject.Value(self.get_column_type(column), value)
RuntimeError: maximum recursion depth exceeded
[1]15366 segmentation fault  d-feet


There is also a popup that appears before the crash, with the following
content:

org.freedesktop.systemd1 : g-dbus-error-quark:
GDBus.Error:org.freedesktop.DBus.Error.LimitsExceeded: The maximum number of
pending replies per connection has been reached (8)


I can also reproduce this very same problem on a fresh Jessie 8.1 install.



-- System Information:
Debian Release: stretch/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.1.3-jpleau (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages d-feet depends on:
ii  dconf-gsettings-backend [gsettings-backend]  0.24.0-2
ii  gir1.2-glib-2.0  1.44.0-1+b2
ii  gir1.2-gtk-3.0   3.16.6-1
ii  hicolor-icon-theme   0.13-1
ii  python   2.7.9-1
ii  python-gi3.16.2-1

d-feet recommends no packages.

d-feet suggests no packages.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#789451: RFS: ags/0.4.2-71-1 [ITP] -- Advanced Gtk+ Sequencer

[Jason Pleau]

Hi


On 06/21/2015 01:07 AM, Joël Krähemann wrote:

[...]

 
 Changes since the last upload:
 
 ags (0.4.2-71-1)
 
   [ Joël Krähemann (Maintainer of Advanced Gtk+ Sequencer) ]
   * commented-out two unimplemented menu entries in editor's machine 
 selector. Add and remove tab you may do so by using resize tab of machine 
 properties dialog.
   * implemented select-all in editor
   * fixed multi-channel editing mode
   * fixed refresh of GUI after cut or paste
   * fixed crash related to XML IO
 
 Regards,
Joël Krähemann
 

The changelog should only list one version entry for new packages (those
not currently in the Debian archive).

Ideally, just the 'Closes: #xx' line, anything else is irrelevant
for a first upload.

If upstream updates its software before you get someone to sponsor your
package, you can simply bump the version number in your d/changelog
file to reflect this.

Also, please fix the lintian warnings shown in
http://mentors.debian.net/package/ags (you can use lintian directly to
see and fix those warnings, before uploading to mentors)

Cheers

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#781054: packaging python-dockerpty

[Jason Pleau]

Hi.

The package is not under DPMT's git repositories on alioth, I've asked
for a review of the package. You can still try it with docker-compose
see if it works for you :)

https://anonscm.debian.org/git/python-modules/packages/dockerpty.git
git://anonscm.debian.org/python-modules/packages/dockerpty.git
ssh://git.debian.org/git/python-modules/packages/dockerpty.git

Also, it seems most packages on python-modules use the tarball directly
from PyPi.org (and thus don't get the unit tests that come with it since
those tarballs only contains the library itself).

I've done the same thing, so packaging the other two libraries shouldn't
be necessary.


-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#781054: packaging python-dockerpty

[Jason Pleau]

Hi !

On 18/05/15 09:19 AM, Felipe Sateler wrote:
 On 17 May 2015 at 20:07, Jason Pleau ja...@jpleau.ca wrote:
 retitle 781054 ITP: python-dockerpty -- pseudo-tty handler for docker
 Python client
 owner 781054 ja...@jpleau.ca
 thanks

 --

 Hi Felipe,

 I'll package this to help you with docker-compose. I have a working
 package right now, it seems to work well!
 
 Great! If you need sponsoring I am available to do that.

The package is available (in git):

http://git.jpleau.ca/debian/python/dockerpty.git/

git clone git://git.jpleau.ca/debian/python/dockerpty.git

I'm currently waiting to join the Debian Python Modules Team so the
package can be maintained there (python-docker is there as well, I
thought it would make sense for dockerpty to be.)

If you want to review/sponsor it right away I can move it to
collab-maint, remove DPMT from Maintainer field and update Vcs-* fields
until I can get it to the python-modules team.

Personnally, I think waiting to get it under DMPT is the better idea..

What's your opinion on that?

 

 There might be more work required to get the package into Debian:
 dockerpty's test suites require behave and expects, two python
 modules that are not in Debian right now.

 Will see what I can do about those too :)
 
 Maybe we can have a non-testsuite-enabled dockerpty in Debian and then
 upload the missing dependencies and enable the suite?
 
 My offer for sponsorship extends to these python modules as well.
 

Works for me !

I have disabled tests for now, and removed the build-depends on them.

Cheers

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#784165: ITP: lua-messagepack -- a pure Lua implementation of MessagePack

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: lua-messagepack
  Version : 0.3.2
  Upstream Author : Francois Perrad francois.per...@gadz.org
* URL : http://fperrad.github.io/lua-MessagePack/
* License : MIT
  Programming Lang: Lua
  Description : a pure Lua implementation of MessagePack
 This module provides an API to convert Lua data structures to MessagePack, and
vice versa.
 .
 MessagePack is an efficient binary serialization format.
 .
 It lets you exchange data among multiple languages like JSON but it's faster.
 and smaller.


This package is a dependency of neovim (#752264)

I plan to maintain it in pkg-lua.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#784058: ITP: lua-dkjson -- JSON module written in Lua

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: lua-dkjson
  Version : 2.5
  Upstream Author : David Heiko Kolf da...@dkolf.de
* URL : http://dkolf.de/src/dkjson-lua.fsl/home
* License : MIT
  Programming Lang: Lua
  Description : JSON module written in Lua
 dkjson is a JSON (JavaScript Object Notation) encoder / decoder. It supports
UTF-8.
 .
 It is written in Lua without any dependencies, but when LPeg (provided in
Debian by lua-lpeg) is installed dkjson uses it to speed up decoding.


This package is a dependency of busted (#783044), which in turn is a dependency
to run tests for neovim (#752264).

I plan to maintain it in pkg-lua.

Additionnal note: This Lua library is currently shipped / embedded with 3 other
packages: vlc, ntopng and sysdig [1]. Once this package is included in this
archive I plan to contact the maintainers of this package to see what could be
done to depend on lua-dkjson, instead of shipping it directly with their
packages.

[1]: http://codesearch.debian.net/perpackage-results/dkjson/2/page_0


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#784103: ITP: lua-mediator -- Event management in Lua for clean and decoupled code

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: lua-mediator
  Version : 1.1.1-0
  Upstream Author : Olivine Labs, LLC.
* URL : https://github.com/Olivine-Labs/mediator_lua
* License : MIT
  Programming Lang: Lua
  Description : Event management in Lua for clean and decoupled code
 mediator_lua is a simple class that allows you to listen to events by
subscribing to and sending data to channels.
 .
 Its purpose is to help you decouple code where you might otherwise have
functions calling functions calling functions.


This package is a dependency of busted (#783044), which in turn is a dependency
to run tests for neovim (#752264).

I plan to maintain it in pkg-lua.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#783667: ITP: lua-cliargs -- command-line argument parsing module for Lua

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: lua-cliargs
  Version : 2.5-1
  Upstream Author : Ahmad Amireh ah...@amireh.net
* URL : https://github.com/amireh/lua_cliargs
* License : MIT
  Programming Lang: Lua
  Description : command-line argument parsing module for Lua

 cliargs is a command-line argument parser for Lua. It supports several types
of arguments:
 .
  * required arguments
  * optional arguments with different notations: -short-key VALUE and/or
--expanded-key=VALUE
  * optional arguments with multiple-values that get appended to a list
  * optional flag arguments (on/off options) with notations: -short-key
and/or --expanded-key
  * a single optional splat argument which can be repeated (must be the last
argument)
 .
 Optional arguments can have default values (strings), flags always default to
'true'.


This package is a dependency of busted (#783044), which in turn is a dependency
to run tests for neovim (#752264).

I plan to maintain it in pkg-lua.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#783302: RFS: entityx/1.1.2-1 [ITP] -- fast, type-safe C++ Entity Component System

[Jason Pleau]

Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for reviews (and possibly a sponsor) for my package entityx:

* Package name: entityx
  Version : 1.1.2-1
  Upstream Author : Alec Thomas a...@swapoff.org
* URL : https://github.com/alecthomas/entityx
* License : Expat
  Section : libs

It builds those binary packages:

  libentityx-doc   : fast, type-safe C++ Entity Component System (API
documentation)

  libentityx-dev   : fast, type-safe C++ Entity Component System
(development files)

  libentityx1  : fast, type-safe C++ Entity Component System

  libentityx1-dbg  : Debug symbols for libentityx1


To access further information about this package, please visit the
following URL:

  http://mentors.debian.net/package/entityx


Alternatively, one can download the package with dget using this command:

  dget -x
http://mentors.debian.net/debian/pool/main/e/entityx/entityx_1.1.2-1.dsc

For people who prefer working with git, there is also a git repository
of the packaging files available at
https://anonscm.debian.org/cgit/collab-maint/entityx.git -- one can
clone it using

  git clone git://anonscm.debian.org/collab-maint/entityx.git


More information about entityx can be obtained from
https://github.com/alecthomas/entityx

Changes since the last upload:

 * Initial release (Closes: #772414)

Additionnal notes:

1.

I added a lintian-override for the symbols file in libentityx1. The
generated symbols were different between i386 and amd64, and since I
can't test for other architectures myself I couldn't tell if they were
different on say armel for example. Any help on that is appreciated.

2.

To make the build reproducible, I had to use a newer version of
dh-strip-nondeterminism than what's available in unstable right now.
(This was to remove timestamps from PNG files)

I built the git version available here:
https://anonscm.debian.org/cgit/reproducible/strip-nondeterminism.git (I
assume it'll get uploaded to unstable after jessie release).

This means my package cannot be included in Debian right now, but I'd be
happy if someone could review my package until dh-strip-determinism is
in unstable.

Thanks !

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#783215: ITP: lua-say -- Lua string hashing/indexing library

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: lua-say
  Version : 1.3-1
  Upstream Author : 2012 Olivine Labs, LLC.
* URL : https://github.com/Olivine-Labs/say/
* License : MIT
  Programming Lang: Lua
  Description : Lua string hashing/indexing library

say is a simple string key/value store for i18n or any other case where you
want namespaced strings.

It is a dependency of busted (#783044), which in turn is a dependency of
neovim (#752264).

I plan to maintain it in pkg-lua


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#783218: ITP: lua-luassert -- Lua assertions extension

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: lua-luassert
  Version : 1.7.61
  Upstream Author : Olivine Labs, LLC.
* URL : https://github.com/Olivine-Labs/luassert/
* License : MIT
  Programming Lang: Lua
  Description : Lua assertions extension

 luassert extends Lua's built-in assertions to provide additional tests and
 the ability to create your own.
 .
 It comes preloaded with argument formatters for common Lua types, but it is
 easy to add custom ones as well.

It is a dependency of busted (#783044), which in turn is a dependency to run
tests for neovim (#752264).

I plan to maintain it in pkg-lua


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#783216: ITP: lua-ansicolors -- simple Lua function for printing to the console in color

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: lua-ansicolors
  Version : 1.0.2
  Upstream Author : Enrique García kik...@gmail.com
* URL : https://github.com/kikito/ansicolors.lua/
* License : MIT
  Programming Lang: Lua
  Description : simple Lua function for printing to the console in color

 ansicolors provides the 'colors' function which allows the Lua programmer to
 output colored (both foreground and background) and stylized text.
 .
 One can use standard terminal colors: black, red, green, yellow, blue,
 magenta, cyan, white.
 .
 The following styles are also available: bright, dim, underline, blink,
 reverse, hidden, reset.

It is a dependency of busted (#783044), which in turn is a dependency to run
tests for neovim (#752264).

I plan to maintain it in pkg-lua


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#782533: [Pkg-mpd-maintainers] Bug#782533: #769951 still exists

[Jason Pleau]

Hello

On 13/04/15 03:20 PM, Joey Hess wrote:
 Package: mpd
 Version: 0.19.1-1.1
 Severity: normal
 
 I just experienced a bug identical to #769951 while running
 the version of mpd that claims to have fixed it. 
 
 I followed the documentation in /etc/default/mpd and disabed it by
 running sudo update-rc.d mpd disable
 
 After reboot, init was listening on :6600 until I ran:
 sudo systemctl  disable mpd.socket
 
 This was on a system freshly installed from stable and upgraded to
 unstable today.
 
[...]

As far as I can tell, bug #769951 was fixed in experimental, but not in
unstable (Likely due to jessie freeze)

Version in experimental as of today is 0.19.9-1

Cheers

Jason


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#781958: rofi: man page doesn't say how to activate/deactivate daemon mode

[Jason Pleau]

Hi Jason !

On 05/04/15 10:59 AM, Jason Woofenden wrote:
 Package: rofi
 Version: 0.15.2-1
 Severity: minor
 
 Thanks for packaging rofi!
 
 I wanted to give it a try, so I installed it with aptitude, and
 took a look at the man page. The man page explains that rofi can be
 run as a daemon (like xbindkeys) or as a one-off, like dmenu.
 
 I looked over the man-page fairly thoroughly, and I can't see
 anything about how you specify which of these behaviors you would
 like.
 
 I would like to see explicit indication of:
 
 1.Whether the default is daemon, or one-off.
 
 2.What commandline option(s) activate daemon mode.
 
 3.What commandline option(s) de-activate daemon mode.
 
 
 I've, as of yet, had no success getting rofi to do anything that I
 can detect. I've tried some sensible commandlines:
 
   rofi
 
   rofi --help
 
   rofi -dmenu
 
   rofi -dmenu -show run
 
 In all of the above cases, rofi seems to hang, doing nothing (maybe
 it's in daemon mode?)


I fully agree with you that the documentation could be better. I have
been working with upstream to improve this, and a lot of work has been
done already on git: http://github.com/DaveDavenport/rofi

 
 additionally, this command:
 
   rofi -h
 
 Prints the following error:
 
   man: /usr/share/man/man1/rofi.1: No such file or directory
 
 

This was also fixed upstream, I'll keep in touch with the developer to
upload a new version to Debian soon !


 Thanks!
 

Thank you for reporting this :)


Cheers

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#774744: ITP: obs -- Open Broadcast Software

[Jason Pleau]

Hi

On Wed, 07 Jan 2015 02:57:45 +0100 =?utf-8?q?Carl_F=C3=BCrstenberg?=
azat...@gmail.com wrote:
 Package: wnpp
 Severity: wishlist
 Owner: Carl Fürstenberg azat...@gmail.com
 
 * Package name: obs
   Version : 0.7.2
   Upstream Author : Hugh Bailey obs@gmail.com
 * URL : https://obsproject.com/
 * License : GPL
   Programming Lang: C, C++
   Description : Open Broadcast Software
 
 a rewrite of what was formerly known as Open Broadcaster Software, 
 software originally designed for recording and streaming live
 video content, efficiently. 
 
 

Just wondering if you had started working on the packaging for
obs-studio ? The Linux version is now officially out ! :)

I am currently using OBS to stream to twitch using the source package
available at upstream's PPA:
https://launchpad.net/~obsproject/+archive/ubuntu/obs-studio. I rebuilt
the package myself though ;)

But I would prefer using the future official Debian package for this! I
think libobs would need its own package for example, the ubuntu ppa
packs everything into a single big binary package.

Hopefully we can get obs-studio in Debian in a near future !

Cheers

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Hi

On 14/03/15 05:52 AM, Paul Wise wrote:
 On Sat, Mar 14, 2015 at 4:47 PM, Stefano Zacchiroli wrote:
 On Sat, Mar 14, 2015 at 09:33:55AM +0800, Paul Wise wrote:
 On Fri, 2015-03-13 at 19:32 +0100, Stefano Zacchiroli wrote:
 This has now been fixed, so I'm closing this bug.

 I think the suites should redirect to the right version, they don't
 yet.

 Please elaborate.
 
 Compare the URL you get when visiting these two URLs:
 
 https://sources.debian.net/src/network-manager/experimental/
 https://sources.debian.net/src/network-manager/latest/
 
 Jason has already implemented the change and is in the process of
 testing it before submitting.
 

Here's the patch :)

I fixed two issues in one commit, since they were on the very same part
of the code: The one reported by Paul, and the other that a package with
multiple versions in the same suite (example, glibc) would not get
redirected to the latest version when going to /src/package/suite/

Thanks

-- 
Jason Pleau
From b7a42373874defca8eb2330be7ed9a7b33b858d3 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sun, 15 Mar 2015 18:33:02 -0400
Subject: [PATCH] suites/redirects: rework the way redirects were done

 * Do HTTP redirects instead of just rendering the page for the
   correct version.

 * Fixed an issue where packages with multiple versions would not
   redirect to the correct versions when going to /src/package/suite/.

   Example: a package in sid could have version 2.0-1 for amd64, and version
   1.0-1 for i386. Going to /src/package/sid/ should redirect us to
   /src/package/2.0-1/, and not /src/package/1.0-1/.
---
 debsources/app/sources/views.py | 30 ++
 debsources/tests/test_webapp.py | 10 ++
 2 files changed, 28 insertions(+), 12 deletions(-)

diff --git a/debsources/app/sources/views.py b/debsources/app/sources/views.py
index 28d2b2e..ac36912 100644
--- a/debsources/app/sources/views.py
+++ b/debsources/app/sources/views.py
@@ -255,6 +255,16 @@ class SourceView(GeneralView):
 pkg_infos=pkg_infos
 )
 
+def _redirect_to_url(self, redirect_url):
+if self.d.get('api'):
+self.render_func = bind_redirect(url_for('.api_source',
+ path_to=redirect_url))
+else:
+self.render_func = bind_redirect(url_for('.source',
+ path_to=redirect_url))
+
+return dict(redirect=redirect_url)
+
 def _handle_latest_version(self, package, path):
 
 redirects to the latest version for the requested page,
@@ -275,14 +285,7 @@ class SourceView(GeneralView):
 else:
 redirect_url = '/'.join([package, version, path])
 
-if self.d.get('api'):
-self.render_func = bind_redirect(url_for('.api_source',
- path_to=redirect_url))
-else:
-self.render_func = bind_redirect(url_for('.source',
- path_to=redirect_url))
-
-return dict(redirect=redirect_url)
+return self._redirect_to_url(redirect_url)
 
 def get_objects(self, path_to):
 
@@ -316,9 +319,12 @@ class SourceView(GeneralView):
 session, package)
 except InvalidPackageOrVersionError:
 raise Http404Error(%s not found % package)
-for version_suite in versions_w_suites:
-if version in version_suite['suites']:
-return self._render_location(
-package, version_suite['version'], path)
+
+versions = sorted([v['version'] for v in versions_w_suites
+  if version in v['suites']],
+  cmp=version_compare)
+if versions:
+redirect_url = '/'.join([package, versions[-1]])
+return self._redirect_to_url(redirect_url)
 
 return self._render_location(package, version, path)
diff --git a/debsources/tests/test_webapp.py b/debsources/tests/test_webapp.py
index 6118b68..b83dba3 100644
--- a/debsources/tests/test_webapp.py
+++ b/debsources/tests/test_webapp.py
@@ -417,6 +417,16 @@ class DebsourcesTestCase(unittest.TestCase, DbTestFixture):
  follow_redirects=True).data)
 self.assertIn(2.03-2, rv['path'])
 
+def test_multiple_versions_in_suite(self):
+rv = json.loads(self.app.get('/api/src/patch/sid/',
+ follow_redirects=True).data)
+self.assertIn('2.7.5-1', rv['path'])
+
+def test_multiple_versions_in_suite_alias(self):
+rv = json.loads(self.app.get('/api/src/patch/unstable/',
+ follow_redirects=True).data)
+self.assertIn('2.7.5-1', rv['path'])
+
 def test_codesearch_box(self

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Updated patches to use the latest testdata commit

Cheers

Jason
From 16d72a6e13a6cada34215dda3a6d4946a772f232 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sun, 16 Nov 2014 11:27:43 -0500
Subject: [PATCH 1/2] allow redirects to package versions based on
 suite/codename

We now check for suite names to redirect to a specific version. If package foo
has version 1.5-1 in sid, going to /src/foo/sid will display as if we went to
/src/foo/1.5-1.

How this works: If we go to /src/foo/sid for example, we iterate through
the package's (foo) suites. If we find 'sid', we simply handle the
request as if it was /src/foo/1.5-1.

This commit also adds a suites_aliases table which let us setup aliases to
suites (ie: unstable-sid, testing-jessie). I have also updated the
updater.py script to also auto-generate those aliases, by parsing the
symlinks in the dists/ folders.

When a user requests /src/foo/unstable, it will lookup the suites_aliases
table and check if this alias is mapped to a suite. If yes, it will handle
the request as if the user requested that particular suite (in this case,
/src/foo/sid). Which in turns would handle it as if it was /src/foo1.5-1.
(taking the example above)
---
 debsources/app/sources/views.py   | 19 ++-
 debsources/debmirror.py   | 24 
 debsources/migrate/007-to-008.sql | 10 ++
 debsources/models.py  | 18 +++---
 debsources/tests/test_webapp.py   | 10 ++
 debsources/updater.py | 14 ++
 6 files changed, 87 insertions(+), 8 deletions(-)
 create mode 100644 debsources/migrate/007-to-008.sql

diff --git a/debsources/app/sources/views.py b/debsources/app/sources/views.py
index e6c6bf1..21b5770 100644
--- a/debsources/app/sources/views.py
+++ b/debsources/app/sources/views.py
@@ -10,7 +10,8 @@ from debsources.excepts import (
 from debsources.consts import SLOCCOUNT_LANGUAGES
 from debsources import statistics
 from debsources.models import (
-PackageName, SourceFile, Checksum, Directory, Location, SuiteInfo)
+PackageName, SourceFile, Checksum, Directory, Location, SuiteInfo,
+SuiteAlias)
 
 from ..views import GeneralView, app, session
 from ..extract_stats import extract_stats
@@ -305,4 +306,20 @@ class SourceView(GeneralView):
 if version == latest:  # we search the latest available version
 return self._handle_latest_version(package, path)
 else:
+# Check if an alias was used. If positive, handle the request
+# as if the suite was used instead of the alias
+check_for_alias = session.query(SuiteAlias) \
+.filter(SuiteAlias.alias == version).first()
+if check_for_alias:
+version = check_for_alias.suite
+try:
+versions_w_suites = PackageName.list_versions_w_suites(
+session, package)
+except InvalidPackageOrVersionError:
+raise Http404Error(%s not found % package)
+for version_suite in versions_w_suites:
+if version in version_suite['suites']:
+return self._render_location(
+package, version_suite['version'], path)
+
 return self._render_location(package, version, path)
diff --git a/debsources/debmirror.py b/debsources/debmirror.py
index 0ce81ba..a8cfa1a 100644
--- a/debsources/debmirror.py
+++ b/debsources/debmirror.py
@@ -273,6 +273,30 @@ class SourceMirror(object):
 
 return suites
 
+def ls_suites_with_aliases(self):
+ list suites, as well as their aliases
+
+Return value: { suite: [aliases] }
+Example: { sid: [unstable], jessie: [testing] }
+
+
+suites = {}
+
+def add_suite(suite):
+if suite not in suites:
+suites[suite] = []
+
+for f in os.listdir(self._dists_dir):
+path = os.path.join(self._dists_dir, f)
+if os.path.isdir(path):
+if not os.path.islink(path):
+add_suite(f)
+else:
+add_suite(os.readlink(path))
+suites[os.readlink(path)].append(f)
+
+return suites
+
 
 class SourceMirrorArchive(SourceMirror):
 Handle for a local Debian source mirror archive, i.e. a mirror of
diff --git a/debsources/migrate/007-to-008.sql b/debsources/migrate/007-to-008.sql
new file mode 100644
index 000..ba647c7
--- /dev/null
+++ b/debsources/migrate/007-to-008.sql
@@ -0,0 +1,10 @@
+CREATE TABLE suites_aliases (
+  alias VARCHAR NOT NULL,
+  suite VARCHAR NOT NULL,
+  CONSTRAINT suites_aliases_alias_suite_key 
+UNIQUE(alias),
+  CONSTRAINT suites_aliases_suite_fkey
+FOREIGN KEY (suite) REFERENCES suites_info(name)
+ON DELETE CASCADE,
+  PRIMARY KEY (alias, suite)
+);
diff --git a/debsources

Bug#762944: debsources: make .pc/ exclusion a configuration parameter

[Jason Pleau]

Hello again,

As discussed on IRC, I have rebased my patches for this bug.


Cheers

-- 
Jason Pleau
From ee67677ce389a844221e3a712598147faf1ff2ff Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Mon, 9 Mar 2015 12:34:06 -0400
Subject: [PATCH 1/2] add a new hidden_files setting in config.ini

Right now the .pc/ exclusion in directory listings is hardcoded. This
new hidden_files configuration allows us to be more flexible in what is
shown or hidden in those listings.

*/.pc/ is added as a default value for this configuration setting.

Closes #762944
---
 .../sources/templates/sources/source_folder.html   | 48 --
 debsources/app/sources/views.py|  6 +--
 debsources/mainlib.py  |  4 +-
 debsources/models.py   | 18 +---
 debsources/tests/test_webapp.py| 22 ++
 doc/examples/sample-config.local.ini   | 12 ++
 etc/config.ini | 12 ++
 7 files changed, 89 insertions(+), 33 deletions(-)

diff --git a/debsources/app/sources/templates/sources/source_folder.html b/debsources/app/sources/templates/sources/source_folder.html
index ba6f894..ac3b5b4 100644
--- a/debsources/app/sources/templates/sources/source_folder.html
+++ b/debsources/app/sources/templates/sources/source_folder.html
@@ -32,31 +32,35 @@
   /tr
 
 {% for dir in subdirs %}
-  tr
-td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/places/folder.png alt=d  //td
-{% if config[DIR_LS_LONG] %}td class=stat-typespan{{ dir.stat.type }}/span/td{% endif %}
-td class=stat-permsspan{{ dir.stat.perms }}/span/td
-{% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(dir.stat.size) }}/span/td{% endif %}
-td class=item-namea href={{ url_for('.source', path_to=path+'/'+dir.name) }}{{ dir.name }}/a
-  {% if config[DIR_LS_LONG] %}
-{% if dir.stat.symlink_dest is not none %}{{  →  + dir.stat.symlink_dest }}{% endif %}
-  {% endif %}
-/td
-  /tr
+  {% if not dir['hidden'] %}
+tr
+  td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/places/folder.png alt=d  //td
+  {% if config[DIR_LS_LONG] %}td class=stat-typespan{{ dir.stat.type }}/span/td{% endif %}
+  td class=stat-permsspan{{ dir.stat.perms }}/span/td
+  {% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(dir.stat.size) }}/span/td{% endif %}
+  td class=item-namea href={{ url_for('.source', path_to=path+'/'+dir.name) }}{{ dir.name }}/a
+{% if config[DIR_LS_LONG] %}
+  {% if dir.stat.symlink_dest is not none %}{{  →  + dir.stat.symlink_dest }}{% endif %}
+{% endif %}
+  /td
+/tr
+  {% endif %}
 {% endfor %}
 
 {% for file_ in subfiles %}
-  tr
-td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/mimetypes/ascii.png alt=-  //td
-{% if config[DIR_LS_LONG] %}td class=stat-typespan{{ file_.stat.type }}/span/td{% endif %}
-td class=stat-permsspan{{ file_.stat.perms }}/span/td
-{% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(file_.stat.size) }}/span/td{% endif %}
-td class=item-namea href={{ url_for('.source', path_to=path+'/'+file_.name) }}{{ file_.name }}/a
-  {% if config[DIR_LS_LONG] %}
-{% if file_.stat.symlink_dest is not none %}{{  →  + file_.stat.symlink_dest }}{% endif %}
-  {% endif %}
-/td
-  /tr
+  {% if not file_['hidden'] %}
+tr
+  td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/mimetypes/ascii.png alt=-  //td
+  {% if config[DIR_LS_LONG] %}td class=stat-typespan{{ file_.stat.type }}/span/td{% endif %}
+  td class=stat-permsspan{{ file_.stat.perms }}/span/td
+  {% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(file_.stat.size) }}/span/td{% endif %}
+  td class=item-namea href={{ url_for('.source', path_to=path+'/'+file_.name) }}{{ file_.name }}/a
+{% if config[DIR_LS_LONG] %}
+  {% if file_.stat.symlink_dest is not none %}{{  →  + file_.stat.symlink_dest }}{% endif %}
+{% endif %}
+  /td
+/tr
+  {% endif %}
 {% endfor %}
 /table
 
diff --git a/debsources/app/sources/views.py b/debsources/app/sources/views.py
index e6c6bf1..72c30b4 100644
--- a/debsources/app/sources/views.py
+++ b/debsources/app/sources/views.py
@@ -137,10 +137,8 @@ class SourceView(GeneralView):
 
 renders a directory, lists subdirs and subfiles
 
-directory = Directory(location, toplevel=(location.get_path() == ))
-
-# (if path == , then the dir is toplevel, and we don't want
-# the .pc directory)
+hidden_files = app.config['HIDDEN_FILES'].split( )
+directory = Directory(location, hidden_files)
 
 pkg_infos = Infobox(session, location.get_package(),
 location.get_version()).get_infos()
diff --git a/debsources/mainlib.py b/debsources/mainlib.py
index 45a206b

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Hello !

As discussed on IRC, I rebase my patches for this bug.

I also fixed a few PEP8 issues found by flake8

Note: The redirects won't work until the app is updated with
bin/debsources-update (which I guess is already setup in a crontask in
production?)


Cheers

-- 
Jason Pleau
From d51a5d1060d6716b405b3e8880ba9ae20944af54 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sun, 16 Nov 2014 11:29:55 -0500
Subject: [PATCH 2/2] update doc/db-schema from previous commit

Adds information about the new suites_aliases table
---
 doc/db-schema/debsources.dia| 198 --
 doc/db-schema/debsources.dot|   3 +
 doc/db-schema/debsources.html   | 118 +++-
 doc/db-schema/debsources.neato  |   3 +
 doc/db-schema/debsources.xml| 112 +++
 doc/db-schema/debsources.zigzag.dia | 210 ++--
 6 files changed, 620 insertions(+), 24 deletions(-)

diff --git a/doc/db-schema/debsources.dia b/doc/db-schema/debsources.dia
index 5487cb0..2cc668b 100644
--- a/doc/db-schema/debsources.dia
+++ b/doc/db-schema/debsources.dia
@@ -3178,6 +3178,156 @@
 dia:real val=6.6005/
   /dia:attribute
   dia:attribute name=name
+dia:string#suites_aliases#/dia:string
+  /dia:attribute
+
+  dia:attribute name=comment
+dia:string##/dia:string
+  /dia:attribute
+  dia:attribute name=abstract
+dia:boolean val=false/
+  /dia:attribute
+  dia:attribute name=suppress_attributes
+dia:boolean val=false/
+  /dia:attribute
+  dia:attribute name=suppress_operations
+dia:boolean val=false/
+  /dia:attribute
+  dia:attribute name=visible_attributes
+dia:boolean val=true/
+  /dia:attribute
+  dia:attribute name=visible_comments
+dia:boolean val=true/
+  /dia:attribute
+  dia:attribute name=wrap_operations
+dia:boolean val=false/
+  /dia:attribute
+  dia:attribute name=wrap_after_char
+dia:int val=40/
+  /dia:attribute
+  dia:attribute name=line_color
+dia:color val=#00/
+  /dia:attribute
+  dia:attribute name=fill_color
+dia:color val=#ff/
+  /dia:attribute
+  dia:attribute name=text_color
+dia:color val=#00/
+  /dia:attribute
+  dia:attribute name=normal_font
+dia:font family=monospace style=0 name=Courier/
+  /dia:attribute
+  dia:attribute name=abstract_font
+dia:font family=monospace style=88 name=Courier/
+  /dia:attribute
+  dia:attribute name=polymorphic_font
+dia:font family=monospace style=8 name=Courier/
+  /dia:attribute
+  dia:attribute name=classname_font
+dia:font family=sans style=80 name=Helvetica/
+  /dia:attribute
+  dia:attribute name=abstract_classname_font
+dia:font family=sans style=88 name=Helvetica/
+  /dia:attribute
+  dia:attribute name=comment_font
+dia:font family=sans style=8 name=Helvetica/
+  /dia:attribute
+  dia:attribute name=font_height
+dia:real val=0.80004/
+  /dia:attribute
+  dia:attribute name=polymorphic_font_height
+dia:real val=0.80004/
+  /dia:attribute
+  dia:attribute name=abstract_font_height
+dia:real val=0.80004/
+  /dia:attribute
+  dia:attribute name=classname_font_height
+dia:real val=1/
+  /dia:attribute
+  dia:attribute name=abstract_classname_font_height
+dia:real val=1/
+  /dia:attribute
+  dia:attribute name=comment_font_height
+dia:real val=1/
+  /dia:attribute
+  dia:attribute name=attributes
+
+dia:composite type=umlattribute
+  dia:attribute name=name
+dia:string#  alias#/dia:string
+  /dia:attribute
+  dia:attribute name=type
+dia:string#character varying#/dia:string
+  /dia:attribute
+  dia:attribute name=value
+
+dia:string/
+
+  /dia:attribute
+  dia:attribute name=visibility
+dia:enum val=3/
+  /dia:attribute
+  dia:attribute name=abstract
+dia:boolean val=false/
+  /dia:attribute
+  dia:attribute name=class_scope
+dia:boolean val=false/
+  /dia:attribute
+/dia:composite
+
+dia:composite type=umlattribute
+  dia:attribute name=name
+dia:string#  suite#/dia:string
+  /dia:attribute
+  dia:attribute name=type
+dia:string#character varying#/dia:string
+  /dia:attribute
+  dia:attribute name=value
+
+dia:string/
+
+  /dia:attribute
+  dia:attribute name=visibility
+dia:enum val=3/
+  /dia:attribute
+  dia:attribute name=abstract
+dia:boolean val=false/
+  /dia:attribute

Bug#762944: debsources: make .pc/ exclusion a configuration parameter

[Jason Pleau]

Hello Stefano,

I have attached two patches. One fixes the issues we talked about
earlier today, and the other implements the Show / Hide hidden files
button in directory listing:

On 10/03/15 09:43 AM, Stefano Zacchiroli wrote:
 On Mon, Mar 09, 2015 at 02:59:54PM -0400, Jason Pleau wrote:
 Updated patch that fixes an error if hidden_files is empty
 
 Hey Jason, thanks a lot for your patch! It looks great in general. I
 just have a few minor changes to request, if you don't mind. See below
 for details.
 
 Right now the .pc/ exclusion in directory listings is hardcoded. This
 new hidden_files configuration allows us to be more flexible in what is
 shown or hidden in those listings.

 */.pc/ is added as a default value for this configuration setting.
 
 This is one concern I have. You've modified the sample config file, but
 not the default configuration stored in mainlib.py. I haven't tested
 that, but I suspect that for that reason upgrading a currently deployed
 Debsources instance to a version that include your patch will trigger
 failures at runtime, if config.local.ini isn't updated. Can you confirm
 that?
 
 If so, I'd very much prefer changing DEFAULT_CONFIG in mainlib.py to
 include */.pc/ as default setting to 1) avoid breakages, and 2) retain
 the current behavior.

As talked on IRC, I have added */.pc/ to DEFAULT_CONFIG.web_app.

 
 +# space-separated list of files or directories patterns to hide in 
 +# directory listings
 +hidden_files: */.pc/
 
 It would be nice to document in the comment here (for lack of a better
 place...) the intended semantics of hidden_files, i.e.: match on the
 full path, relative to which dir, and maybe the fact that fnmatch() is
 involved, for reference.
 
I have added a bit more comments, feel free to adjust if you think you
can phrase it better than it is currently (I have no shame in admitting
that I may sometimes not be clear enough in English :) )

 There are also a couple of minor flake8 issues in your patch (which I
 can fix upon patch import, if needed).

They now seem fixed on my side, you can re-run flake8 to confirm !

 
 
 And from your other email:
 This also allow us to add a toggle button (show / hide hidden files) in
 directory listings eventually. I could do that in a separate commit in
 this bug if you'd like.
 
 Yes please, that would be awesome! :)
 

Done :) See 0002 patch !

 
 Cheers.
 

Thanks

-- 
Jason Pleau
From 641a210eb0d06dfc63a9332891981bcfd07e3426 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Tue, 10 Mar 2015 20:37:22 -0400
Subject: [PATCH 2/2] source_folder: add a show/hide hidden files button

Since that hidden files and directories are still passed to the view,
but are now given an 'hidden' boolean attribute, we can show or hide
them once the page is loaded.

I also changed the way javascript was loaded in debsources.js, so that
it does not run unnecessary code (for example we don't need to print
line numbers in folders).
---
 debsources/app/static/css/source_folder.css|  13 ++
 debsources/app/static/javascript/debsources.js | 273 ++---
 debsources/app/templates/source_file.html  |   2 +-
 debsources/app/templates/source_folder.html|  59 +++---
 debsources/app/views.py|   1 +
 5 files changed, 196 insertions(+), 152 deletions(-)

diff --git a/debsources/app/static/css/source_folder.css b/debsources/app/static/css/source_folder.css
index 1a45b8f..1bd9c6e 100644
--- a/debsources/app/static/css/source_folder.css
+++ b/debsources/app/static/css/source_folder.css
@@ -40,3 +40,16 @@ License: GNU Affero General Public License, version 3 or above.
 font-size: small;
 text-align: right;
 }
+
+.hidden_file {
+display: none;
+}
+
+.hidden_file.visible {
+display: table-row;
+}
+
+#btn_toggle_hidden_files {
+	font-size: 0.8em;
+}
+
diff --git a/debsources/app/static/javascript/debsources.js b/debsources/app/static/javascript/debsources.js
index 844e58a..ced5cf6 100644
--- a/debsources/app/static/javascript/debsources.js
+++ b/debsources/app/static/javascript/debsources.js
@@ -1,4 +1,4 @@
-/* Copyright (C) 2014  Jason Pleau ja...@jpleau.ca
+/* Copyright (C) 2015  Jason Pleau ja...@jpleau.ca
  *
  * This file is part of Debsources.
  *
@@ -27,128 +27,151 @@
  *
  */
 
-var debsources = function(message_pos) {
-var print_lines = function() {
-var position = message_pos;
-var msgbox = document.getElementById('messages');
-var index = document.getElementById('sourceslinenumbers');
-var divHeight = msgbox.offsetHeight;
-var lineHeight = parseInt(window.getComputedStyle(index).getPropertyValue('line-height'),10);
-var lines = Math.ceil(divHeight / lineHeight)+1; // always insert one more line below the last line of code
-
-for(i=0; ilines; ++i){
-var element = document.createElement('a');
-var s = 'a/abr'; // lines corr. messages do no need indexes
-element.innerHTML

Bug#762944: debsources: make .pc/ exclusion a configuration parameter

[Jason Pleau]

Hi Stefano.

I have a bit on this bug, please see the attached patch.

Instead of filtering out files that should be hidden, I have added an
'hidden' (bool) key to the dictionary that it returns, so that the API
can return all the files, even if they're hidden. It should not be the
model nor the controller that decides what is shown on the screen, but
the view.

This also allow us to add a toggle button (show / hide hidden files) in
directory listings eventually. I could do that in a separate commit in
this bug if you'd like.

Cheers !

-- 
Jason Pleau
From 6b2b89b6e63baa089f4f671286a69cc2b22a915f Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Mon, 9 Mar 2015 12:34:06 -0400
Subject: [PATCH] add a new hidden_files setting in config.ini

Right now the .pc/ exclusion in directory listings is hardcoded. This
new hidden_files configuration allows us to be more flexible in what is
shown or hidden in those listings.

*/.pc/ is added as a default value for this configuration setting.

Closes #762944
---
 debsources/app/templates/source_folder.html | 48 -
 debsources/app/views.py |  5 ++-
 debsources/models.py| 18 +++
 debsources/tests/test_webapp.py | 21 +
 doc/examples/sample-config.local.ini|  4 +++
 etc/config.ini  |  4 +++
 6 files changed, 69 insertions(+), 31 deletions(-)

diff --git a/debsources/app/templates/source_folder.html b/debsources/app/templates/source_folder.html
index 444194a..99ca1c1 100644
--- a/debsources/app/templates/source_folder.html
+++ b/debsources/app/templates/source_folder.html
@@ -32,31 +32,35 @@
   /tr
 
 {% for dir in subdirs %}
-  tr
-td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/places/folder.png alt=d  //td
-{% if config[DIR_LS_LONG] %}td class=stat-typespan{{ dir.stat.type }}/span/td{% endif %}
-td class=stat-permsspan{{ dir.stat.perms }}/span/td
-{% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(dir.stat.size) }}/span/td{% endif %}
-td class=item-namea href={{ url_for('source_html', path_to=path+'/'+dir.name) }}{{ dir.name }}/a
-  {% if config[DIR_LS_LONG] %}
-{% if dir.stat.symlink_dest is not none %}{{  →  + dir.stat.symlink_dest }}{% endif %}
-  {% endif %}
-/td
-  /tr
+  {% if not dir['hidden'] %}
+tr
+  td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/places/folder.png alt=d  //td
+  {% if config[DIR_LS_LONG] %}td class=stat-typespan{{ dir.stat.type }}/span/td{% endif %}
+  td class=stat-permsspan{{ dir.stat.perms }}/span/td
+  {% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(dir.stat.size) }}/span/td{% endif %}
+  td class=item-namea href={{ url_for('source_html', path_to=path+'/'+dir.name) }}{{ dir.name }}/a
+{% if config[DIR_LS_LONG] %}
+  {% if dir.stat.symlink_dest is not none %}{{  →  + dir.stat.symlink_dest }}{% endif %}
+{% endif %}
+  /td
+/tr
+  {% endif %}
 {% endfor %}
 
 {% for file_ in subfiles %}
-  tr
-td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/mimetypes/ascii.png alt=-  //td
-{% if config[DIR_LS_LONG] %}td class=stat-typespan{{ file_.stat.type }}/span/td{% endif %}
-td class=stat-permsspan{{ file_.stat.perms }}/span/td
-{% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(file_.stat.size) }}/span/td{% endif %}
-td class=item-namea href={{ url_for('source_html', path_to=path+'/'+file_.name) }}{{ file_.name }}/a
-  {% if config[DIR_LS_LONG] %}
-{% if file_.stat.symlink_dest is not none %}{{  →  + file_.stat.symlink_dest }}{% endif %}
-  {% endif %}
-/td
-  /tr
+  {% if not file_['hidden'] %}
+tr
+  td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/mimetypes/ascii.png alt=-  //td
+  {% if config[DIR_LS_LONG] %}td class=stat-typespan{{ file_.stat.type }}/span/td{% endif %}
+  td class=stat-permsspan{{ file_.stat.perms }}/span/td
+  {% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(file_.stat.size) }}/span/td{% endif %}
+  td class=item-namea href={{ url_for('source_html', path_to=path+'/'+file_.name) }}{{ file_.name }}/a
+{% if config[DIR_LS_LONG] %}
+  {% if file_.stat.symlink_dest is not none %}{{  →  + file_.stat.symlink_dest }}{% endif %}
+{% endif %}
+  /td
+/tr
+  {% endif %}
 {% endfor %}
 /table
 
diff --git a/debsources/app/views.py b/debsources/app/views.py
index 2519c5f..6a99377 100644
--- a/debsources/app/views.py
+++ b/debsources/app/views.py
@@ -554,10 +554,9 @@ class SourceView(GeneralView):
 
 renders a directory, lists subdirs and subfiles
 
-directory = Directory(location, toplevel=(location.get_path() == ))
 
-# (if path == , then the dir is toplevel, and we don't want
-# the .pc directory)
+hidden_files = app.config

Bug#762944: debsources: make .pc/ exclusion a configuration parameter

[Jason Pleau]

Updated patch that fixes an error if hidden_files is empty

Sorry for the noise :)
From 275b94fe8031ac81c75388190dcd58709b14f8d7 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Mon, 9 Mar 2015 12:34:06 -0400
Subject: [PATCH] add a new hidden_files setting in config.ini

Right now the .pc/ exclusion in directory listings is hardcoded. This
new hidden_files configuration allows us to be more flexible in what is
shown or hidden in those listings.

*/.pc/ is added as a default value for this configuration setting.

Closes #762944
---
 debsources/app/templates/source_folder.html | 48 -
 debsources/app/views.py |  5 ++-
 debsources/models.py| 18 +++
 debsources/tests/test_webapp.py | 21 +
 doc/examples/sample-config.local.ini|  4 +++
 etc/config.ini  |  4 +++
 6 files changed, 69 insertions(+), 31 deletions(-)

diff --git a/debsources/app/templates/source_folder.html b/debsources/app/templates/source_folder.html
index 444194a..99ca1c1 100644
--- a/debsources/app/templates/source_folder.html
+++ b/debsources/app/templates/source_folder.html
@@ -32,31 +32,35 @@
   /tr
 
 {% for dir in subdirs %}
-  tr
-td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/places/folder.png alt=d  //td
-{% if config[DIR_LS_LONG] %}td class=stat-typespan{{ dir.stat.type }}/span/td{% endif %}
-td class=stat-permsspan{{ dir.stat.perms }}/span/td
-{% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(dir.stat.size) }}/span/td{% endif %}
-td class=item-namea href={{ url_for('source_html', path_to=path+'/'+dir.name) }}{{ dir.name }}/a
-  {% if config[DIR_LS_LONG] %}
-{% if dir.stat.symlink_dest is not none %}{{  →  + dir.stat.symlink_dest }}{% endif %}
-  {% endif %}
-/td
-  /tr
+  {% if not dir['hidden'] %}
+tr
+  td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/places/folder.png alt=d  //td
+  {% if config[DIR_LS_LONG] %}td class=stat-typespan{{ dir.stat.type }}/span/td{% endif %}
+  td class=stat-permsspan{{ dir.stat.perms }}/span/td
+  {% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(dir.stat.size) }}/span/td{% endif %}
+  td class=item-namea href={{ url_for('source_html', path_to=path+'/'+dir.name) }}{{ dir.name }}/a
+{% if config[DIR_LS_LONG] %}
+  {% if dir.stat.symlink_dest is not none %}{{  →  + dir.stat.symlink_dest }}{% endif %}
+{% endif %}
+  /td
+/tr
+  {% endif %}
 {% endfor %}
 
 {% for file_ in subfiles %}
-  tr
-td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/mimetypes/ascii.png alt=-  //td
-{% if config[DIR_LS_LONG] %}td class=stat-typespan{{ file_.stat.type }}/span/td{% endif %}
-td class=stat-permsspan{{ file_.stat.perms }}/span/td
-{% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(file_.stat.size) }}/span/td{% endif %}
-td class=item-namea href={{ url_for('source_html', path_to=path+'/'+file_.name) }}{{ file_.name }}/a
-  {% if config[DIR_LS_LONG] %}
-{% if file_.stat.symlink_dest is not none %}{{  →  + file_.stat.symlink_dest }}{% endif %}
-  {% endif %}
-/td
-  /tr
+  {% if not file_['hidden'] %}
+tr
+  td class=item-imgimg src={{ config['ICONS_FOLDER'] }}22x22/mimetypes/ascii.png alt=-  //td
+  {% if config[DIR_LS_LONG] %}td class=stat-typespan{{ file_.stat.type }}/span/td{% endif %}
+  td class=stat-permsspan{{ file_.stat.perms }}/span/td
+  {% if config[DIR_LS_LONG] %}td class=stat-sizespan{{ {:,d}.format(file_.stat.size) }}/span/td{% endif %}
+  td class=item-namea href={{ url_for('source_html', path_to=path+'/'+file_.name) }}{{ file_.name }}/a
+{% if config[DIR_LS_LONG] %}
+  {% if file_.stat.symlink_dest is not none %}{{  →  + file_.stat.symlink_dest }}{% endif %}
+{% endif %}
+  /td
+/tr
+  {% endif %}
 {% endfor %}
 /table
 
diff --git a/debsources/app/views.py b/debsources/app/views.py
index 2519c5f..6a99377 100644
--- a/debsources/app/views.py
+++ b/debsources/app/views.py
@@ -554,10 +554,9 @@ class SourceView(GeneralView):
 
 renders a directory, lists subdirs and subfiles
 
-directory = Directory(location, toplevel=(location.get_path() == ))
 
-# (if path == , then the dir is toplevel, and we don't want
-# the .pc directory)
+hidden_files = app.config['HIDDEN_FILES'].split( )
+directory = Directory(location, hidden_files)
 
 pkg_infos = Infobox(session,
 location.get_package(),
diff --git a/debsources/models.py b/debsources/models.py
index e4d05fc..de015db 100644
--- a/debsources/models.py
+++ b/debsources/models.py
@@ -19,6 +19,7 @@
 import os
 import magic
 import stat
+import fnmatch
 from collections import namedtuple
 
 from sqlalchemy import Column, ForeignKey
@@ -652,11

Bug#779545: ITP: rofi -- A window switcher, run dialog and dmenu replacement

[Jason Pleau]

Hello !

On 03/03/15 03:59 PM, Alessandro Ghedini wrote:
 On dom, mar 01, 2015 at 11:42:00 -0500, Jason Pleau wrote:
 Package: wnpp
 Severity: wishlist
 Owner: Jason Pleau ja...@jpleau.ca

 * Package name: rofi
   Version : 0.15.2
   Upstream Author : Dave Davenport qb...@gmpclient.org
 * URL : https://davedavenport.github.io/rofi/
 * License : MIT/X11
   Programming Lang: C
   Description : A window switcher, run dialog and dmenu replacement

 rofi can act as an application launcher, window switcher, ssh launcher and
 dmenu replacement.

 [...]

 I plan on maintaining this package in collab-maint.
 
 Hello,
 
 what's the status on this? I'm available for sponsoring, if needed.
 
 Cheers
 

Packaging is going well, there are a few things I'd like to talk about
with upstream before asking for a review/sponsor:

 * make test fails with the release tarball

 * dpkg-shlibdeps warns me about unused dependencies, will need to
investigate a bit

 * gcc gives a few warnings while compiling, would be nice to have those
fixed I think

 * a few typos to fix in the source code


I will push my files to collab-maint by friday (and will let you know!)

Thanks!


-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#779545: ITP: rofi -- A window switcher, run dialog and dmenu replacement

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: rofi
  Version : 0.15.2
  Upstream Author : Dave Davenport qb...@gmpclient.org
* URL : https://davedavenport.github.io/rofi/
* License : MIT/X11
  Programming Lang: C
  Description : A window switcher, run dialog and dmenu replacement

rofi can act as an application launcher, window switcher, ssh launcher and
dmenu replacement.

Included features:

- Full keyboard navigation

- Type to filter (case insensitive)

- RTL language support

- UTF-8 enabled

- Optional history based ordering (last 25 choices are ordered on top based on
use)

- Can be extended using scripts

- Optional use of Levenshtein distance to order matches while typing


This package does is not a dependency of any package in the Debian archives
yet.

I plan on maintaining this package in collab-maint.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#779471: [debhelper-devel] Bug#779471: dh_installchangelogs: please add CHANGES.md as an upstream changelog

[Jason Pleau]

Hello Axel,

Thanks for the quick reply!


On 01/03/15 04:06 AM, Axel Beckert wrote:
 Hi Jason,

 
 Jason Pleau wrote:
 Several projects that are hosted on Github use markdown files (.md) as their
 README and changelogs.
 
 Yeah, you also see .markdown and .mdwn as suffixes. Not as common as
 .md, but still common.
 
 At least the usage of README.md already caused confusion and bug
 reports in Debian, see https://bugs.debian.org/678630 -- but since
 dh_installdocs doesn't have such an automatism, there's no need to
 discuss the best solution for that.
 
 CHANGES.md is used by a few projects, and it would be nice to have this file
 automatically picked up as an upstream changelog.
 
 I fully agree and have committed and pushed your changes. Thanks for
 your contribution.

Awesome, thanks :)

 
 I've also checked for occurrences of CHANGES.mdwn, CHANGES.markdown,
 changelog.mdwn and changelog.markdown and found a few:

 
 CHANGES.mdwn   = 0 Packages: 
 http://codesearch.debian.net/results/path%3ACHANGES.mdwn%20%3D%3D%3D/page_0
 CHANGES.markdown   = 1 Package:  
 http://codesearch.debian.net/results/path%3ACHANGES.markdown%20%3D%3D%3D/page_0
 ChangeLog.mdwn = 0 Packages: 
 http://codesearch.debian.net/results/path%3AChangeLog.mdwn%20%3D%3D%3D/page_0
 ChangeLog.markdown = 0 Packages: 
 http://codesearch.debian.net/results/path%3AChangeLog.markdown%20%3D%3D%3D/page_0
 HISTORY.mdwn   = 0 Packages: 
 http://codesearch.debian.net/results/path%3AHISTRORY.mdwn%20%3D%3D%3D/page_0
 HISTORY.markdown   = 0 Packages: 
 http://codesearch.debian.net/results/path%3AHISTRORY.markdown%20%3D%3D%3D/page_0
 
 (I also checked some other capitalizations of ChangeLog and
 HISTORY which seemed common or at least possible to me. No hits
 there either.)
 
 Then I searched a little bit with DuckDuckGo and found indeed some
 more occurrences, even by an DD:
 
 https://duckduckgo.com/?q=%22ChangeLog.mdwn%22+-ikiwiki
 
 history.mdwn also has a few hits:
 
 https://duckduckgo.com/?q=%22HISTORY.mdwn%22+-ikiwiki+-git-annex
 
 CHANGES.markdown even a few more:
 
 https://duckduckgo.com/?q=%22CHANGES.markdown%22+-ikiwiki+-git-annex+-recentchanges
 
 Another common suffix for such files seem to be rst (IIRC
 ReStructuredText coming from the Python world):
 
 https://duckduckgo.com/?q=%22CHANGES.rst%22
 http://codesearch.debian.net/results/path%3ACHANGES.rst%20hange/page_0
 
 The latter lists 60 current packages. So I'll soon add .rst as
 changelog suffix to look for, but I'm rather hesitating to add .mdwn
 or .markdown.
 
   Regards, Axel
 

Good catch on those extra files. I would personally add .markdown, but
not .mdwn (I did not even know this was a thing !)

Cheers

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#779471: dh_installchangelogs: please add CHANGES.md as an upstream changelog

[Jason Pleau]

Package: debhelper
Version: 9.20150101
Severity: wishlist
Tags: patch

Dear Maintainer,

Several projects that are hosted on Github use markdown files (.md) as their
README and changelogs.

CHANGES.md is used by a few projects, and it would be nice to have this file
automatically picked up as an upstream changelog.

Trivial patch (against 9.20150101) is included to add this file to
dh_installchangelogs



-- System Information:
Debian Release: 8.0
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.19.0-jpleau (SMP w/8 CPU cores)
Locale: LANG=en_CA.UTF-8, LC_CTYPE=en_CA.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
Init: systemd (via /run/systemd/system)

Versions of packages debhelper depends on:
ii  binutils  2.25-5
ii  dpkg  1.17.24
ii  dpkg-dev  1.17.24
ii  file  1:5.22+15-1
ii  libdpkg-perl  1.17.24
ii  man-db2.7.0.2-5
ii  perl  5.20.1-5
ii  po-debconf1.0.16+nmu3

debhelper recommends no packages.

Versions of packages debhelper suggests:
ii  dh-make  1.20140617
From e07cd5a706caa8f742dd416cad26e8044a78fb5c Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sat, 28 Feb 2015 22:52:33 -0500
Subject: [PATCH] dh_installchangelogs: Add CHANGES.md to the list of common
 changelog filenames

---
 dh_installchangelogs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dh_installchangelogs b/dh_installchangelogs
index edd7f9f..df0f2c2 100755
--- a/dh_installchangelogs
+++ b/dh_installchangelogs
@@ -142,7 +142,7 @@ if (! defined $upstream) {
 	if (! isnative($dh{MAINPACKAGE})  !compat(6)) {
 		foreach my $dir (qw{. doc docs}) {
 			my @files=sort glob($dir/*);
-			foreach my $name (qw{changelog changes changelog.txt changes.txt history history.txt changelog.md}) {
+			foreach my $name (qw{changelog changes changelog.txt changes.txt history history.txt changelog.md changes.md}) {
 my @matches=grep {
 	lc basename($_) eq $name  -s $_  ! excludefile($_)
 } @files;
-- 
2.1.4

Bug#774808: debsources: line number undefined

[Jason Pleau]

Hello again.

I was wrong in my previous email, the code I mentionned is not the cause
of the bug in question. It was an error that I did not catch when
testing with Firefox.

Matthieu, can you test the attached patch and see if it fixes the issue
for you?

Cheers

Jason
From 181b7b6e6f57966b534f352c61a8e7cc81572b7d Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Thu, 8 Jan 2015 22:51:43 -0500
Subject: [PATCH] web app: fix 'undefined' line numbers in source code pages

Problem: In Iceweasel / Firefox, clicking on a line-number when browsing
a source file caused the line number returned to be undefined, and
so #Lundefined appeared in the URL.

The cause was that to retrieve the line numbers from the a elements we
clicked on, I was using .innerText, which is not supported by Firefox /
Iceweasel. The alternative is using .textContent.

The solution was to use element.textContent || element.innerText, so
that it works browsers that support either .textContent or .innerText.

Closes #774808
---
 debsources/app/static/javascript/debsources.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/debsources/app/static/javascript/debsources.js b/debsources/app/static/javascript/debsources.js
index 6c8d2ee..844e58a 100644
--- a/debsources/app/static/javascript/debsources.js
+++ b/debsources/app/static/javascript/debsources.js
@@ -115,10 +115,10 @@ var debsources = function(message_pos) {
 
 if (!event.shiftKey || !last_clicked) {
 last_clicked = callerElement;
-change_hash_without_scroll(callerElement, L + callerElement.innerText);
+change_hash_without_scroll(callerElement, L + (callerElement.textContent || callerElement.innerText));
 } else {
-var first_line = parseInt(last_clicked.innerText);
-var second_line = parseInt(callerElement.innerText);
+var first_line = parseInt(last_clicked.textContent || last_clicked.innerText);
+var second_line = parseInt(callerElement.textContent || callerElement.innerText);
 
 if (second_line  first_line) {
 var tmp = first_line;
-- 
2.1.4

Bug#774808: debsources: line number undefined

[Jason Pleau]

Hello!

On 08/01/15 10:31 AM, Matthieu Caneill wrote:
 On 7 January 2015 at 20:43, Reiner Herrmann rei...@reiner-h.de wrote:
 I'm using Iceweasel 34.0, and when I'm viewing a source file
 on sources.debian.net and click on a line number, it jumps
 to the current URL with #Lundefined appended (instead of e.g. #L123).

 It is working correctly with konqueror, but it occurs even with
 a new iceweasel profile (i.e. no addons installed).
 
 Hi,
 Thanks for the report.
 I can reproduce this bug. As it works fine when I manually enter such
 an URL, I believe the bug is due to the new javascript module
 (debsources.js) which automatically highlights lines with internal
 links (#L42).
 Jason: any idea where this comes from?
 

I will need to have a more closer look tonight after work, but at first
sight what I can notice is that there is a javascript error in the page
(in debsources.js at line 35). This part of the file was added by this
commit [1]

---
33: var msgbox = document.getElementById('messages');
34: var index = document.getElementById('sourceslinenumbers');
35: var divHeight = msgbox.offsetHeight;
---

If msgbox is null (because there are no elements with an ID of
'messages', then msgbox.offsetHeight will error out.


Sometimes a javascript error like this can lead to weird behaviors,
similar to what Reiner reported. But I will test a bit more tonight to
confirm if that's the case or not (and try to submit a patch for the
little issue I pointed above).



 Cheers,
 --
 Matthieu
 

Thanks

Jason

[1]:
http://anonscm.debian.org/cgit/qa/debsources.git/commit/debsources/app/static/javascript/debsources.js?id=e246fcfeb7f20dde3f026d2c4ff8f7e71fc8e303


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#772414: ITP: entityx -- A fast, type-safe C++ Entity Component System

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau oxez@goldeneye

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: entityx
  Version : 1.0.1
  Upstream Author : Alec Thomas a...@swapoff.org
* URL : https://github.com/alecthomas/entityx
* License : MIT
  Programming Lang: C++
  Description : A fast, type-safe C++ Entity Component System


EntityX is an Entity Component system that uses C++11 features to provide type-
safe component management, event delivery, etc. It was built with game
development in mind, although it can be used with all kinds of projects.

I use this library and would like to see it in the Debian archive. It's a
wonderful tool that allows developers to focus on their project, instead of
having to re-invent the wheel for this kind of programming pattern.

I plan to package it in collab-maint.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Hello Stefano,

I have attached two patches: One for the generated db-schema, and the
other for the actual code modifications.

You'll see that there's now a suites_aliases table, so that we can
handle multiple aliases per suites.

I updated the updater.py script to also save those aliases, but I
didn't do it the way we talked about previously; instead of using
Release files, I wrote something similar to
debmirror.SourceMirror.ls_suites(), making use of symlinks to map
aliases to suites.


There is something I'm not sure about: I get 3 failing tests. They don't
seem related to my patches. At first sight it looks like the assertions
haven't been updated to the latest test-data, but I could be wrong..

Here's the output:

==
FAIL: areaFiltersMatchReferenceDb (debsources.tests.test_stats.Stats)
--
Traceback (most recent call last):
  File /mnt/data/dev/debsources/debsources/tests/test_stats.py, line
178, in areaFiltersMatchReferenceDb
self.assertEqual(area_count, 43622)
AssertionError: 43810L != 43622

==
FAIL: ctagsCountsMatchReferenceDb (debsources.tests.test_stats.Stats)
--
Traceback (most recent call last):
  File /mnt/data/dev/debsources/debsources/tests/test_stats.py, line
133, in ctagsCountsMatchReferenceDb
self.assertSuiteCountsEqual(ctags, statistics.ctags)
  File /mnt/data/dev/debsources/debsources/tests/test_stats.py, line
47, in assertSuiteCountsEqual
(expected_count, actual_count, suite))
AssertionError: 21395 != 21483 for suite jessie

==
FAIL: statsMatchReferenceDb (debsources.tests.test_updater.MetadataCache)
--
Traceback (most recent call last):
  File /mnt/data/dev/debsources/debsources/tests/test_updater.py, line
319, in statsMatchReferenceDb
self.assertDictContainsSubset(expected_stats, self.stats)
AssertionError: Mismatched values: 'total.ctags', expected: 70166,
actual: 70354,'debian_sid.ctags', expected: 21395, actual:
21483,'debian_squeeze.ctags', expected: 30633, actual: 30721






Cheers

-- 
Jason Pleau
From 4b11bca1805ac0041220e5d1a5d21189dd4eef21 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sun, 16 Nov 2014 11:27:43 -0500
Subject: [PATCH 1/2] allow redirects to package versions based on
 suite/codename

We now check for suite names to redirect to a specific version. If package foo
has version 1.5-1 in sid, going to /src/foo/sid will display as if we went to
/src/foo/1.5-1.

How this works: If we go to /src/foo/sid for example, we iterate through
the package's (foo) suites. If we find 'sid', we simply handle the
request as if it was /src/foo/1.5-1.

This commit also adds a suites_aliases table which let us setup aliases to
suites (ie: unstable-sid, testing-jessie). I have also updated the
updater.py script to also auto-generate those aliases, by parsing the
symlinks in the dists/ folders.

When a user requests /src/foo/unstable, it will lookup the suites_aliases
table and check if this alias is mapped to a suite. If yes, it will handle
the request as if the user requested that particular suite (in this case,
/src/foo/sid). Which in turns would handle it as if it was /src/foo1.5-1.
(taking the example above)
---
 debsources/app/views.py   | 18 +-
 debsources/debmirror.py   | 24 
 debsources/migrate/007-to-008.sql | 10 ++
 debsources/models.py  | 18 +++---
 debsources/tests/test_webapp.py   | 10 ++
 debsources/updater.py | 14 ++
 6 files changed, 86 insertions(+), 8 deletions(-)
 create mode 100644 debsources/migrate/007-to-008.sql

diff --git a/debsources/app/views.py b/debsources/app/views.py
index 565b17a..fc3cc14 100644
--- a/debsources/app/views.py
+++ b/debsources/app/views.py
@@ -37,7 +37,7 @@ from debsources.excepts import (
 Http500Error, Http404Error, Http404ErrorSuggestions, Http403Error)
 from debsources.models import (
 Ctag, Package, PackageName, Checksum, Location, Directory,
-SourceFile, File)
+SourceFile, File, SuiteAlias)
 from debsources.app.sourcecode import SourceCodeIterator
 from debsources.app.forms import SearchForm
 from debsources.app.infobox import Infobox
@@ -572,6 +572,22 @@ class SourceView(GeneralView):
 if version == latest:  # we search the latest available version
 return self._handle_latest_version(package, path)
 else:
+# Check if an alias was used. If positive, handle the request as if
+# the suite was used instead of the alias
+check_for_alias = session.query(SuiteAlias

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Hello Stefano,

Thanks for your feedback on this. I know how time consuming it can be to
review someone else's code when they're new to a project, so please know
I do appreciate it !

I have a couple of comments / questions to some of your suggestions :

On 13/11/14 04:37 PM, Stefano Zacchiroli wrote:
 
 diff --git a/debsources/migrate/007-to-008.sql 
 b/debsources/migrate/007-to-008.sql
 new file mode 100644
 index 000..838346e
 --- /dev/null
 +++ b/debsources/migrate/007-to-008.sql
 @@ -0,0 +1,14 @@
 +ALTER TABLE suites_info
 +  ADD COLUMN alias VARCHAR;
 
 Regarding this, which is an important design decision here, I'm not
 particularly happy about having a maximum of 1 alias per suite. Why
 couldn't a suite have more?
 
 Now, implement this properly in SQL would require a separate table to
 allow 1:N suite-aliases mappings, which is probably a tad overkill for
 what we need here. So how about instead: 1/ renaming the new column to
 aliases, and 2/ document that it is a comma-separated list of aliases,
 that all map to the current row in the suites_info table?
 
 Of course your changes to views.py and models.py will need to be adapted
 accordingly.
 

The idea of having multiple aliases per suite never crossed my mind. I'm
not sure it exists right now in Debian but that's a good idea.

The logical thing would be to create a separate table to allow the
association between a suite and its aliases. I think it would ease the
rest of the code since we'd let SQLAlchemy handle creating and saving
the 'aliases' array.

However if you're 100% certain that there will never be a need to link
specific data to a suite+alias association (I can't think of any),
storing a comma-separated string of aliases in the 'suites' table would
be fine for me.

 +UPDATE suites_info
 +  SET alias='unstable' WHERE name='sid';
 +
 +UPDATE suites_info
 +  SET alias='testing' WHERE name='jessie';
 +
 +UPDATE suites_info
 +  SET alias='stable' WHERE name='wheezy';
 +
 +UPDATE suites_info
 +  SET alias='oldstable' WHERE name='squeeze';
 
 This is another part of the design that bothers me a little. The above
 is OK for deploying quickly the change, but I certainly do not want to
 introduce another place in Debsources which will have to be manually
 maintained/updated at each Debian release.
 
 So what I think we should do here is modifying the Debsources updated to
 retrieve aliases from Release files, and re-fill the suites_info table
 (including aliases information) at each update run. What do you think?
 

Agreed about having the suites and their aliases auto-updated.

I remember we talked about it a bit on #debian-qa with pabs, there was
the possibility to use an API once it's finished:
https://api.ftp-master.debian.org/suites

If the API is not close to release, I can look into parsing Release
files, are you aware of a parsing tool in python? I was looking at
python-apt, but it seems to rely on a sources.list file, I don't think
that's not what we need.

 @@ -194,15 +194,17 @@ class SuiteInfo(Base):
  version = Column(String, nullable=True)
  release_date = Column(Date, nullable=True)
  sticky = Column(Boolean, nullable=False)
 +alias = Column(String, nullable=True)
 
 If you're OK with my proposed change, it'd be nice to have automatically
 split of the aliases list every time a SuiteInfo object is created, but
 I'm not sure what's the most appropriate way to do that in SQLAlchemy.
 If it's annoying to do, we can certainly live with keeping that as a
 comma-separated string and split it when needed.
 

It would be annoying to have to split the comma-separated string every
time we want to make use of the aliases. I vote for splitting it into a
list (or tuple?) when a SuiteInfo object is instantiated. (That's if we
do it with a string, and not a 1:N mapping table)


 HTH,
 Cheers.
 

Thanks

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Updated patch

I made a proper git patch, so that the new file would be displayed. Also
cleaned up the models.py file, I had some leftover from previous strategies.

It's still required to regenerate the data in testdata (make distclean
dump) to account for the new column

Thanks

-- 
Jason Pleau
From fd6ff45605f17c9e91664e3bf160f23ed1515597 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Tue, 11 Nov 2014 06:52:38 -0500
Subject: [PATCH] allow redirects to package versions based on suite/codename

Like the /latest redirect, we now check for suite names to redirect to
a specific version. If package foo has version 1.5-1 in sid, going to
/src/foo/sid will display as if we went to /src/foo/1.5-1.

When going to /src/foo/sid for example, It iterates through the package's suite,
and if it finds what's provided in the url, it redirects the request (in the
application) to the proper version.

This commit also adds an 'alias' column to the suites_info table. This allows
us to add 'unstable' to 'sid', 'testing' to 'jessie' and so on. We'll be able
to update this column in an update script, and it will be easy for other
distributions using debsources to manage their suites aliases.

When going to /src/foo/unstable, if it detects that 'unstable' matches
'sid', it will change the version to 'sid', and will use the same logic
as described above to redirect to the right version.
---
 debsources/app/views.py | 16 +++-
 debsources/migrate/007-to-008.sql   | 14 ++
 debsources/models.py| 10 ++
 debsources/tests/test_webapp.py | 10 ++
 doc/db-schema/debsources.dia| 23 +++
 doc/db-schema/debsources.dot|  2 +-
 doc/db-schema/debsources.html   | 18 +-
 doc/db-schema/debsources.neato  |  2 +-
 doc/db-schema/debsources.xml| 16 
 doc/db-schema/debsources.zigzag.dia | 23 +++
 10 files changed, 126 insertions(+), 8 deletions(-)
 create mode 100644 debsources/migrate/007-to-008.sql

diff --git a/debsources/app/views.py b/debsources/app/views.py
index 565b17a..3cbafc0 100644
--- a/debsources/app/views.py
+++ b/debsources/app/views.py
@@ -37,7 +37,7 @@ from debsources.excepts import (
 Http500Error, Http404Error, Http404ErrorSuggestions, Http403Error)
 from debsources.models import (
 Ctag, Package, PackageName, Checksum, Location, Directory,
-SourceFile, File)
+SourceFile, File, SuiteInfo)
 from debsources.app.sourcecode import SourceCodeIterator
 from debsources.app.forms import SearchForm
 from debsources.app.infobox import Infobox
@@ -572,6 +572,20 @@ class SourceView(GeneralView):
 if version == latest:  # we search the latest available version
 return self._handle_latest_version(package, path)
 else:
+suites_info = session.query(SuiteInfo).all()
+suites_aliases = { s.alias: s.name for s in suites_info }
+try:
+versions_w_suites = PackageName.list_versions_w_suites(
+session, package)
+except InvalidPackageOrVersionError:
+raise Http404Error(%s not found % package)
+for version_suite in versions_w_suites:
+if version in suites_aliases:
+version = suites_aliases[version]
+if version in version_suite['suites']:
+return self._render_location(
+package, version_suite['version'], path)
+
 return self._render_location(package, version, path)
 
 
diff --git a/debsources/migrate/007-to-008.sql b/debsources/migrate/007-to-008.sql
new file mode 100644
index 000..838346e
--- /dev/null
+++ b/debsources/migrate/007-to-008.sql
@@ -0,0 +1,14 @@
+ALTER TABLE suites_info
+  ADD COLUMN alias VARCHAR;
+
+UPDATE suites_info
+  SET alias='unstable' WHERE name='sid';
+
+UPDATE suites_info
+  SET alias='testing' WHERE name='jessie';
+
+UPDATE suites_info
+  SET alias='stable' WHERE name='wheezy';
+
+UPDATE suites_info
+  SET alias='oldstable' WHERE name='squeeze';
diff --git a/debsources/models.py b/debsources/models.py
index 45647af..86b6648 100644
--- a/debsources/models.py
+++ b/debsources/models.py
@@ -42,8 +42,8 @@ from debsources.consts import SUITES
 Base = declarative_base()
 
 
-# used for migrations, see scripts under python/migrate/
-DB_SCHEMA_VERSION = 7
+# used for migrations, see scripts under debsources/migrate/
+DB_SCHEMA_VERSION = 8
 
 
 class PackageName(Base):
@@ -194,15 +194,17 @@ class SuiteInfo(Base):
 version = Column(String, nullable=True)
 release_date = Column(Date, nullable=True)
 sticky = Column(Boolean, nullable=False)
+alias = Column(String, nullable=True)
 
-def __init__(self, name, sticky=False, version=None, release_date=None):
+def __init__(self, name, sticky=False, version

Bug#767826: python3-django-assets: fails to install: SyntaxError: Missing parentheses in call to 'print'

[Jason Pleau]

Hello,

I have noticed this error when installing this package.

Attached is a trivial patch that fixes this bug

Sent upstream: https://github.com/miracle2k/django-assets/pull/47

-- 
Jason Pleau
Description: python3 compat for pytest_plugin.py
 Trivial change to make the print statement compatible with python3 in the
 django_assets/pytest_plugin.py
 .
 Closes #767826
Author: Jason Pleau ja...@jpleau.ca
Forwarded: https://github.com/miracle2k/django-assets/pull/47
---
This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
--- a/django_assets/pytest_plugin.py
+++ b/django_assets/pytest_plugin.py
@@ -3,5 +3,5 @@
 
 @pytest.fixture(autouse=True)
 def set_django_assets_env():
-print Set django assets environment
+print(Set django assets environment)
 django_assets.env.get_env() # initialise django-assets settings

Bug#769061: ITP: vim-easymotion -- simple and efficient motions in vim

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: vim-easymotion
  Version: 2.0
  Upstream Author: Kim Silkebækken k...@silkebaekken.no, haya14busa
* URL: https://github.com/Lokaltog/vim-easymotion
* License: Creative Commons Attribution Share-Alike (CC-BY-SA) v3.0
  Programming Lang: VimL
  Description: simple and efficient motions in vim

(shameless copy paste from upstream's url)

EasyMotion provides a much simpler way to use some motions in vim. It
takes the number out of numberw or numberf{char} by highlighting
all possible choices and allowing you to press one key to jump directly
to the target.

When one of the available motions is triggered, all visible text
preceding or following the cursor is faded, and motion targets are
highlighted.


I intend to maintain this package in the pkg-vim team.

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Hello !

I have spent some time on this; it seems to work well on my local
install of debsources. However I am not sure how to tackle
stable/testing/unstable aliases to wheezy/jessie/sid respectively.

@ Stefano, do you have a suggestion on how I could achieve this?

For example, /src/linux/stable/ should redirect to /src/linux/wheezy/.
But this should not be hardcoded, since stable will need to redirect to
jessie eventually.


I have attached a patch of my work so far

Cheers

-- 
Jason Pleau
diff --git a/debsources/app/views.py b/debsources/app/views.py
index 565b17a..dc17e3c 100644
--- a/debsources/app/views.py
+++ b/debsources/app/views.py
@@ -572,6 +572,15 @@ class SourceView(GeneralView):
 if version == latest:  # we search the latest available version
 return self._handle_latest_version(package, path)
 else:
+try:
+versions_w_suites = PackageName.list_versions_w_suites(
+session, package)
+except InvalidPackageOrVersionError:
+raise Http404Error(%s not found % package)
+for version_suite in versions_w_suites:
+if version in version_suite['suites']:
+return self._render_location(
+package, version_suite['version'], path)
 return self._render_location(package, version, path)
 
 

Bug#761149: debsources: allow redirects to package versions based on suite/codename

[Jason Pleau]

Hello again,

I have worked a bit more on this (see attached patch) :

 * There's a new column in suites_info: alias. (For example, in the
suite 'sid', alias will be 'unstable')
 * The database schema in doc/ has been updated to reflect the above
 * unstable/testing/stable/oldstable redirects now work (assuming the
alias column is set correctly for each suite)
 * I have added unit tests for both suite redirects (/sid) and alias
redirects (/unstable)

After our discussion this morning on #debian-qa, I thought it'd be a
good idea to include the suites alias in the database directly :

 * It allows other distributions to easily manage them
 * On our side, we'll simply need to update this table in each update
run (calling an external API for example to get the update suites).

   There's no code in this patch to update the table, I think we were
waiting for the API to be finalized?

I'll submit a proper git commit patch once we have ironed out issues :)

Note: The data in testdata/ git submodule must be updated (make
distclean dump) or else the unit tests will fail because of a missing
column.

Thanks

-- 
Jason Pleau
diff --git a/debsources/app/views.py b/debsources/app/views.py
index 565b17a..3cbafc0 100644
--- a/debsources/app/views.py
+++ b/debsources/app/views.py
@@ -37,7 +37,7 @@ from debsources.excepts import (
 Http500Error, Http404Error, Http404ErrorSuggestions, Http403Error)
 from debsources.models import (
 Ctag, Package, PackageName, Checksum, Location, Directory,
-SourceFile, File)
+SourceFile, File, SuiteInfo)
 from debsources.app.sourcecode import SourceCodeIterator
 from debsources.app.forms import SearchForm
 from debsources.app.infobox import Infobox
@@ -572,6 +572,20 @@ class SourceView(GeneralView):
 if version == latest:  # we search the latest available version
 return self._handle_latest_version(package, path)
 else:
+suites_info = session.query(SuiteInfo).all()
+suites_aliases = { s.alias: s.name for s in suites_info }
+try:
+versions_w_suites = PackageName.list_versions_w_suites(
+session, package)
+except InvalidPackageOrVersionError:
+raise Http404Error(%s not found % package)
+for version_suite in versions_w_suites:
+if version in suites_aliases:
+version = suites_aliases[version]
+if version in version_suite['suites']:
+return self._render_location(
+package, version_suite['version'], path)
+
 return self._render_location(package, version, path)
 
 
diff --git a/debsources/models.py b/debsources/models.py
index 45647af..78f32ad 100644
--- a/debsources/models.py
+++ b/debsources/models.py
@@ -42,8 +42,8 @@ from debsources.consts import SUITES
 Base = declarative_base()
 
 
-# used for migrations, see scripts under python/migrate/
-DB_SCHEMA_VERSION = 7
+# used for migrations, see scripts under debsources/migrate/
+DB_SCHEMA_VERSION = 8
 
 
 class PackageName(Base):
@@ -163,24 +163,6 @@ class Package(Base):
 Index('ix_packages_name_id_version', Package.name_id, Package.version)
 
 
-class Suite(Base):
-
-Debian suites (squeeze, wheezy, etc) mapping with source package versions
-
-__tablename__ = 'suites'
-__table_args__ = (UniqueConstraint('package_id', 'suite'),)
-
-id = Column(Integer, primary_key=True)
-package_id = Column(Integer,
-ForeignKey('packages.id', ondelete=CASCADE),
-index=True, nullable=False)
-suite = Column(String, index=True)
-
-def __init__(self, package, suite):
-self.package_id = package.id
-self.suite = suite
-
-
 class SuiteInfo(Base):
 static information about known suites
 
@@ -194,14 +176,38 @@ class SuiteInfo(Base):
 version = Column(String, nullable=True)
 release_date = Column(Date, nullable=True)
 sticky = Column(Boolean, nullable=False)
+alias = Column(String, nullable=True)
 
-def __init__(self, name, sticky=False, version=None, release_date=None):
+def __init__(self, name, sticky=False, version=None, release_date=None,
+ alias=None):
 self.name = name
 if version:
 self.version = version
 if release_date:
 self.release_date = release_date
 self.sticky = sticky
+self.alias = alias
+
+
+class Suite(Base):
+
+Debian suites (squeeze, wheezy, etc) mapping with source package versions
+
+__tablename__ = 'suites'
+__table_args__ = (UniqueConstraint('package_id', 'suite'),)
+
+id = Column(Integer, primary_key=True)
+package_id = Column(Integer,
+ForeignKey('packages.id', ondelete=CASCADE),
+index=True, nullable=False)
+suite = Column(String, index

Bug#727781: zsnes: Fatal error when loading state (DSP-1 ROM)

[Jason Pleau]

Hello !

I've been trying to find a way to fix this issue as well for a few days now.

I managed to get it to work with this following changes (see attached file)

The rename of usr/bin/zsnesd = usr/bin/zsnes could probably be handled
better (maybe with dh_links ?), for now as long as I can load my game
states I'm happy :)

Thoughts on this patch? I'm not sure how adding --enable-debug fixes
this, so it could very well lead to other issues.

Cheers

-- 
Jason Pleau
diff --git a/debian/rules b/debian/rules
index 26facf7..9fd3573 100755
--- a/debian/rules
+++ b/debian/rules
@@ -21,7 +21,7 @@ override_dh_auto_configure:
 	dh_auto_configure --sourcedirectory=src -- \
 		--enable-opengl \
 		--disable-cpucheck --enable-release \
-		force_arch=i486
+		force_arch=i486 --enable-debug
 
 override_dh_clean:
 	rm -f src/Makefile src/cfg.h src/config.h src/config.log src/config.status src/input.h src/makefile.dep src/md.h src/parsegen src/tools/depbuild src/zsnes
@@ -29,6 +29,7 @@ override_dh_clean:
 	dh_clean
 
 override_dh_install:
+	mv debian/tmp/usr/bin/zsnesd debian/tmp/usr/bin/zsnes
 	dh_install
 	for res in 16 32 48 64; do \
 		mkdir -p debian/zsnes/usr/share/icons/hicolor/$${res}x$${res}/apps; \

Bug#761103: debsources: highlight #Lxxx lines by default

[Jason Pleau]

Hello Stefano,

On 07/11/14 12:24 PM, Stefano Zacchiroli wrote:
 On Thu, Nov 06, 2014 at 08:09:15PM -0500, Jason Pleau wrote:
 The attached patch implements your previous suggestions :
 
 Awesome, thanks! Last comments:
 
 1) I've added to the new .js file a copyright header pointing to
you. Can you confirm you're OK with contributing your code under
AGPLv3, as the rest of Debsources?
 
You can find attached a version of your patch that includes the
copyright header (and fixes some whitespace issues that make Git
cry :-))

I'm OK with my code being under this license ! I'll use this copyright
header if I ever need to create new files in the Debsources project.

 
 +change_hash_without_scroll(callerElement, L + 
 callerElement.getAttribute('data-line'));
 +} else {
 +var first_line = 
 parseInt(last_clicked.getAttribute('data-line'));
 +var second_line = 
 parseInt(callerElement.getAttribute('data-line'));
 [...]
 +a id=L{{ i }} href=#L{{ i }} data-line={{ i }}{{ i 
 }}/abr /
 
 2) In the same anti-bloat vein of my previous comments, do we really
need to another attribute here, given that its content is precisely
the same of the text child of a? Can't you just use the data
content of that text node?
 
(Yes, I understand that in the future the a node might contain more
complex markup, but there will always be a text leaf in the DOM tree
that we can use; if not directly a DOM method that return the textual
value of the whole sub-tree.)
 
 What do you think?
 Cheers.


I usually prefer to not rely on elements' innerHTML/innerText content
(that's just a personal style). But I also always only work in LAN
environments, where speed isn't really an issue. I agree that if we're
browsing a 10k+ line files, those extra bytes can add up quickly.

Please see the attached patch for the fixed version.

Thanks for the feedback !

Jason

From c1a218f019368d6ea6cedab2d813073ade685a32 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sat, 1 Nov 2014 10:44:24 -0400
Subject: [PATCH] web app: automatically highlight #Lxx lines

Add automatic line numbers from #Lxx location.hash

It supports #L50 and #L50-L150 (The latter will highlight from line 50 to line
150).

It also changes the hash if we click on a line, and if we click on a second
line holding the SHIFT key, it will highlight the whole range, and update the
hash as well.

Closes: #761103

Signed-off-by: Stefano Zacchiroli z...@upsilon.cc
---
 debsources/app/static/javascript/debsources.js | 137 +
 debsources/app/templates/source_file.html  |   1 +
 debsources/app/templates/source_file_code.inc.html |   5 +-
 3 files changed, 139 insertions(+), 4 deletions(-)
 create mode 100644 debsources/app/static/javascript/debsources.js

diff --git a/debsources/app/static/javascript/debsources.js b/debsources/app/static/javascript/debsources.js
new file mode 100644
index 000..38e3647
--- /dev/null
+++ b/debsources/app/static/javascript/debsources.js
@@ -0,0 +1,137 @@
+/* Copyright (C) 2014  Jason Pleau ja...@jpleau.ca
+ *
+ * This file is part of Debsources.
+ *
+ * Debsources is free software: you can redistribute it and/or modify it under
+ * the terms of the GNU Affero General Public License as published by the Free
+ * Software Foundation, either version 3 of the License, or (at your option)
+ * any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Affero General Public License
+ * for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see http://www.gnu.org/licenses/.
+ */
+
+/*
+ * Highlight line numbers according to data received in the anchor
+ *
+ * Example: file.cpp#L50-L275 will highlight lines 50 to 275.
+ *
+ * There's also support to select one line or a range of lines (by clicking on
+ * a line or shift-clicking on a range of lines). The URL will be updated with
+ * the selection.
+ *
+ */
+
+(function() {
+function highlight_lines(start, end) {
+// First, remove the highlight class from elements that might already have it
+var elements = document.querySelectorAll(span.highlight);
+for (i = 0; i  elements.length; ++i) {
+var element = elements[i];
+element.className = element.className.replace(/\bhighlight\b/, '');
+}
+
+// Then, add the highlight class to elements that contain the lines we want to highlight
+for (i = start; i = end; ++i) {
+var element = document.getElementById(line + i);
+element.className = element.className +  highlight ;
+}
+}
+
+var hash_changed = function(event, scroll) {
+
+event = typeof event !== 'undefined' ? event

Bug#761103: debsources: highlight #Lxxx lines by default

[Jason Pleau]

Hello Stefano,

The attached patch implements your previous suggestions :

On 06/11/14 02:22 PM, Stefano Zacchiroli wrote:
 Hello again, Jason.
 
 To JavaScript hackers lurking here: your comments on this patch are most
 welcome!
 
 Thanks a lot for this patch. It works like a charm on my local
 deployment. As I'm no JavaScript guru, I could use extra eyeballs to
 review it, and I do have some questions for you (see below), but I see
 no reason why this couldn't be integrated.
 
 -a id=L{{ i }} href=#L{{ i }}{{ i }}/abr /
 +a id=L{{ i }} href=#L{{ i }} class=linenumber data-line={{ 
 i }}{{ i }}/abr /
 
 Is it really needed to add class=linenumber to all a elements?
 
 It seems that the JavaScript only uses that class as a selector. So
 can't one instead rely on the id=sourcelinenumbers that is already on
 the parent pre element, and consider all its a children? That would
 avoid bloat in the generated HTML, reduce load time, etc.

Indeed, it was not needed to add a new class. Using the pre element's
#id is a better choice.

 
 +script type=text/javascript
 [...]
 +/script
 
 I'd like to move this JavaScript snippet to a separate .js file, rather
 than re-shipping it every time a file is rendered. Can you change your
 patch to do so?
 
 I suggest to name it something like
 DEBSOURCES_ROOT/debsources/app/static/javascript/debsources.js (so that
 we can use it in the future for other JavaScript-related needs). It will
 then be accessible from the HTML as /static/javascript/debsources.js
 (note: *not* as /javascript/debsources.js).
 

I have added a new file:
DEBSOURCES_ROOT/debsources/app/static/javascript/debsources.js

It is only included in the source_file.html template, there's no need
(yet) to force the download of this javascript file in other templates.

The code is now wrapped in an an anonymous function, to not pollute the
global namespace if this javascript file is ever loaded in other pages.


Thanks


-- 
Jason Pleau
From 8132506a6d178db416f852161e279d367529d8f7 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sat, 1 Nov 2014 10:44:24 -0400
Subject: [PATCH] add automatic line numbers from #Lxx location.hash

It supports #L50 and #L50-L150 (The latter will highlight from line 50
to line 150).

It also changes the hash if we click on a line, and if we click on a
second line holding the SHIFT key, it will highlight the whole range,
and update the hash as well.
---
 debsources/app/static/javascript/debsources.js | 116 +
 debsources/app/templates/source_file.html  |   1 +
 debsources/app/templates/source_file_code.inc.html |   7 +-
 3 files changed, 119 insertions(+), 5 deletions(-)
 create mode 100644 debsources/app/static/javascript/debsources.js

diff --git a/debsources/app/static/javascript/debsources.js b/debsources/app/static/javascript/debsources.js
new file mode 100644
index 000..20b81d8
--- /dev/null
+++ b/debsources/app/static/javascript/debsources.js
@@ -0,0 +1,116 @@
+/* Highlight line numbers according to data received in the anchor
+ * Example: file.cpp#L50-L275 will highlight lines 50 to 275.
+ * 
+ * There's also support to select one line or a range of lines (by clicking on 
+ * a line or shift-clicking on a range of lines). The URL will be updated
+ * with the selection.
+ */
+
+(function() {
+function highlight_lines(start, end) {
+// First, remove the highlight class from elements that might already have it
+var elements = document.querySelectorAll(span.highlight);
+for (i = 0; i  elements.length; ++i) {
+var element = elements[i];
+element.className = element.className.replace(/\bhighlight\b/, '');
+}
+
+// Then, add the highlight class to elements that contain the lines we want to highlight
+for (i = start; i = end; ++i) {
+var element = document.getElementById(line + i);
+element.className = element.className +  highlight ;
+}
+}
+
+var hash_changed = function(event, scroll) {
+
+event = typeof event !== 'undefined' ? event: null;
+scroll = typeof scroll !== 'undefined' ? scroll: false;
+
+// Will match strings like #L15 and #L15-L20
+var regex = /#L(\d+)(-L(\d+))*$/;
+
+var match = regex.exec(window.location.hash);
+if (match != null) {
+var first_line = second_line = null;
+first_line = parseInt(match[1]);
+
+if (typeof match[3] !== 'undefined'  match[3].length  0) {
+second_line = parseInt(match[3]);
+} else {
+second_line = first_line;
+}
+
+// If we get something like #L20-L15, just swap the two line numbers so the loop will work
+if (second_line  first_line) {
+var tmp = first_line;
+first_line = second_line;
+second_line = tmp;
+}
+
+highlight_lines

Bug#764178: debsources: infobox CSS alignment problem with short files

[Jason Pleau]

Hey Christophe

On 05/11/14 07:51 AM, Christophe Siraut wrote:
 Hi,
 
 My commit adds a padding-right to make sure that even if the file has
 one short line, it's content will be left-aligned.
 
 We could instead limit the expansion of the first column, which contains
 the line numbers, see attachment.
 

Looks like that's a better solution ! I had tested with max-width, it
didn't work for me. Should have tested with a simple width :)




Thanks

 Cheers,
 Christophe
 


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#764178: debsources: infobox CSS alignment problem with short files

[Jason Pleau]

Hello Stefano,


On 04/11/14 04:44 AM, Stefano Zacchiroli wrote:
 First of all: thanks a lot for your patch, Jason!
 
 On Fri, Oct 31, 2014 at 07:42:23AM -0400, Jason Pleau wrote:
 From 6cc9f15d51dd35a5afb82a2c3680e3e5dfc0f93b Mon Sep 17 00:00:00 2001
 From: Jason Pleau ja...@jpleau.ca
 Date: Fri, 31 Oct 2014 00:05:26 -0400
 Subject: [PATCH] source_file: fix text overlapping the infobox
 
 I'm no CSS expert, so I'm unable to comment on your patch at the
 moment. Matthieu: can you have a look and comment on Jason's approach at
 fixing #764178.
 
 I've a separate comment though:
 
 When browsing a file's source on sources.debian.net, if the file
 didn't contain enough text its content would overlap onto the infobox
 to the right.
 
 Your commit message essentially restates the bug report, rather than
 explaining how the corresponding change fixes it. The commit message
 should really do the latter, rather than the former.
 

Thanks for the feedback ! I admit my commit doesn't explain what I did
to fix the issue I was fixing.

First, I'd like to note that there were two issues in the original post
(I think that's what brought up the confusion)

 * If the file was too short, the infobox seemed to expand outside
it's container. This was fixed already in
0ed831b256a91287ebfe63c9a52cbbb76816a293 [1] a few weeks ago.

 * If the file contained only one line with only a few characters (for
example, debian/compat is a one liner containing only one character),
the file content was displayed to the right (see my attached picture:
before.png). If the window browser was small, the text even overlapped
over the infobox.

 Particularly in this case, I see no obvious reason why changing the
 right padding of codetable (an horizontal spacing matter) would fix the
 bug (which seems to be a vertical spacing matter). I'm sure it *does*
 fix the bug, but the commit should explain why it does so, so that even
 CSS illiterates as myself could understand the rationale :-)
 
 Jason: do you think you can update your patch to do so?
 

My commit adds a padding-right to make sure that even if the file has
one short line, it's content will be left-aligned. The 450px that I've
written is a bit arbitrary. You can look at it in a more graphical way
in my attachment after.png. The green part is the padding-right that
I've added.

I have attached a new patch, this time with a more descriptive text,
hopefully it will make more sense !


Thanks !


 Many thanks in advance,
 Cheers.
 

[1]:
http://anonscm.debian.org/cgit/qa/debsources.git/commit/?id=0ed831b256a91287ebfe63c9a52cbbb76816a293

-- 
Jason Pleau
From 2570734fd05f369b1435be89bc830b00b7209279 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Tue, 4 Nov 2014 18:36:10 -0500
Subject: [PATCH] source_file: fix the text alignment in short files

When browsing a file containing only short lines (for example debian/compat
files), its content was aligned to the right, very close to the infobox.

This commit adds a padding-right to the pre element containing the
file's content  allowing it to be aligned to the left even on the problematic
files as described above.
---
 debsources/app/static/css/source_file.css  | 4 
 debsources/app/templates/source_file_code.inc.html | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/debsources/app/static/css/source_file.css b/debsources/app/static/css/source_file.css
index 58328bc..02ae1e8 100644
--- a/debsources/app/static/css/source_file.css
+++ b/debsources/app/static/css/source_file.css
@@ -39,6 +39,10 @@ License: GNU Affero General Public License, version 3 or above.
 
 /* LINE NUMBERS */
 
+#codetable #codecontainer{
+padding-right: 450px;
+}
+
 #codetable #sourceslinenumbers{
 text-align: right;
 border-right: 1px solid black;
diff --git a/debsources/app/templates/source_file_code.inc.html b/debsources/app/templates/source_file_code.inc.html
index 9ea5a88..f0a912d 100644
--- a/debsources/app/templates/source_file_code.inc.html
+++ b/debsources/app/templates/source_file_code.inc.html
@@ -23,7 +23,7 @@
 {%- endfor %}/pre
 /td
 td
-  precode id=sourcecode class={% if file_language -%}
+  pre id=codecontainercode id=sourcecode class={% if file_language -%}
 	{{ file_language }}{% else %}no-highlight
 	{%- endif %}{% for (line, highlight) in code -%}
   {% if highlight -%}
-- 
2.1.1



signature.asc
Description: OpenPGP digital signature

Bug#761103: debsources: highlight #Lxxx lines by default

[Jason Pleau]

Hello !

I have attached a patch that solves this bug.

From the commit message:

It supports two formats: #L50 and #L50-L150 (The latter will highlight
from line 50 to line 150).

It also changes the hash if we click on a line, and if we click on a
second line holding the SHIFT key, it will highlight the whole range,
and update the hash as well, allowing one to share the URL.


I have tested in Chromium 38, Iceweasel 31.2, and IE8 (I assume it
should work for later versions of IE)

Cheers

-- 
Jason Pleau
From 8bde6c68437ae6634297cdff321b495ed6e9a052 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Sat, 1 Nov 2014 10:44:24 -0400
Subject: [PATCH] add automatic line numbers from #Lxx location.hash

It supports #L50 and #L50-L150 (The latter will highlight from line 50
to line 150).

It also changes the hash if we click on a line, and if we click on a
second line holding the SHIFT key, it will highlight the whole range,
and update the hash as well.
---
 debsources/app/templates/source_file_code.inc.html | 117 -
 1 file changed, 112 insertions(+), 5 deletions(-)

diff --git a/debsources/app/templates/source_file_code.inc.html b/debsources/app/templates/source_file_code.inc.html
index 9ea5a88..17bd3d0 100644
--- a/debsources/app/templates/source_file_code.inc.html
+++ b/debsources/app/templates/source_file_code.inc.html
@@ -19,17 +19,14 @@
   tr
 td
   pre id=sourceslinenumbers{% for i in range(1, nlines+1) -%}
-a id=L{{ i }} href=#L{{ i }}{{ i }}/abr /
+a id=L{{ i }} href=#L{{ i }} class=linenumber data-line={{ i }}{{ i }}/abr /
 {%- endfor %}/pre
 /td
 td
   precode id=sourcecode class={% if file_language -%}
 	{{ file_language }}{% else %}no-highlight
 	{%- endif %}{% for (line, highlight) in code -%}
-  {% if highlight -%}
-  span class=highlight{{ line }}/span{% else -%}
-  {{ line }}
-  {%- endif %}
+span id=line{{ loop.index }} class=codeline {% if highlight -%} highlight   {%- endif %}{{ line }}/span
   {%- endfor %}/code/pre
 /td
 {% if msg -%}
@@ -46,3 +43,113 @@
 {%- endif %}
   /tr
 /table
+
+script type=text/javascript
+function highlight_lines(start, end) {
+// First, remove the highlight class from elements that already have it
+var elements = document.querySelectorAll(span.highlight);
+for (i = 0; i  elements.length; ++i) {
+var element = elements[i];
+element.className = element.className.replace(/\bhighlight\b/, '');
+}
+
+// Then, add the highlight class to elements that contain the lines we want to highlight
+for (i = start; i = end; ++i) {
+var element = document.getElementById(line + i);
+element.className = element.className +  highlight ;
+}
+}
+
+var hash_changed = function(event, scroll) {
+
+event = typeof event !== 'undefined' ? event: null;
+scroll = typeof scroll !== 'undefined' ? scroll: false;
+
+// Will match strings like #L15 and #L15-20
+var regex = /#L(\d+)(-L(\d+))*$/;
+
+var match = regex.exec(window.location.hash);
+if (match != null) {
+var first_line = second_line = null;
+first_line = parseInt(match[1]);
+
+if (typeof match[3] !== 'undefined'  match[3].length  0) {
+second_line = parseInt(match[3]);
+} else {
+second_line = first_line;
+}
+
+// If we get something like #L20-L15, just swap the two line numbers so the loop will work
+if (second_line  first_line) {
+var tmp = first_line;
+first_line = second_line;
+second_line = tmp;
+}
+
+highlight_lines(first_line, second_line);
+
+if (scroll) {
+window.scroll(0, document.getElementById(L+first_line).offsetTop);
+}
+}
+}
+
+
+function change_hash_without_scroll(element, hash) {
+// This is necessary because when changing window.location.hash, the window will
+// scroll to the element's id if it matches the hash
+var id = element.id;
+element.id = id+'-tmpNoScroll';
+window.location.hash = hash;
+element.id = id;
+}
+
+var last_clicked;
+var line_click_handler = function(event) {
+if (event.preventDefault) {
+event.preventDefault();
+} else {
+event.returnValue = false;
+}
+
+var callerElement = event.target || event.srcElement;
+
+if (!event.shiftKey || !last_clicked) {
+last_clicked = callerElement;
+change_hash_without_scroll(callerElement, L + callerElement.getAttribute('data-line'));
+} else {
+var first_line = parseInt(last_clicked.getAttribute('data-line'));
+var second_line = parseInt(callerElement.getAttribute('data-line'));
+
+if (second_line  first_line) {
+var tmp = first_line;
+first_line = second_line;
+second_line = tmp;
+}
+
+change_hash_without_scroll

Bug#764178: debsources: infobox CSS alignment problem with short files

[Jason Pleau]

Hello again,

I seem to have included the wrong patch in my previous email, please see
the attached file in this email.

Sorry for the noise


-- 
Jason Pleau
From 6cc9f15d51dd35a5afb82a2c3680e3e5dfc0f93b Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Fri, 31 Oct 2014 00:05:26 -0400
Subject: [PATCH] source_file: fix text overlapping the infobox

When browsing a file's source on sources.debian.net, if the file didn't
contain enough text its content would overlap onto the infobox to the
right.

This fixes the issue on at least Chromium 38 and Iceweasel 31.2.

Closes #764178
---
 debsources/app/static/css/source_file.css | 4 
 1 file changed, 4 insertions(+)

diff --git a/debsources/app/static/css/source_file.css b/debsources/app/static/css/source_file.css
index 58328bc..b21938b 100644
--- a/debsources/app/static/css/source_file.css
+++ b/debsources/app/static/css/source_file.css
@@ -39,6 +39,10 @@ License: GNU Affero General Public License, version 3 or above.
 
 /* LINE NUMBERS */
 
+#codetable pre{
+padding-right: 450px;
+}
+
 #codetable #sourceslinenumbers{
 text-align: right;
 border-right: 1px solid black;
-- 
2.1.1

Bug#764178: debsources: infobox CSS alignment problem with short files

[Jason Pleau]

Hello,

I have made this small patch that fixes the text overlapping on the infobox.

The other issue (that the box containing the infobox was not big enough)
seems to have been fixed by 0ed831b256a91287ebfe63c9a52cbbb76816a293 on
October 12th


Cheers

-- 
Jason Pleau
From 97a3760687e0a369ee4aac3c53bbb6eaf2d69882 Mon Sep 17 00:00:00 2001
From: Jason Pleau ja...@jpleau.ca
Date: Fri, 31 Oct 2014 00:05:26 -0400
Subject: [PATCH] source_file: fix text overlapping the infobox

When browsing a file's source on sources.debian.net, if the file didn't
contain enough text its content would overlap onto the infobox to the
right.

This fixes the issue on at least Chromium 38 and Iceweasel 31.2.

---
 debsources/app/static/css/source_file.css | 4 
 1 file changed, 4 insertions(+)

diff --git a/debsources/app/static/css/source_file.css b/debsources/app/static/css/source_file.css
index 58328bc..a70aa30 100644
--- a/debsources/app/static/css/source_file.css
+++ b/debsources/app/static/css/source_file.css
@@ -39,6 +39,10 @@ License: GNU Affero General Public License, version 3 or above.
 
 /* LINE NUMBERS */
 
+#codetable pre{
+padding-right: 50px;
+}
+
 #codetable #sourceslinenumbers{
 text-align: right;
 border-right: 1px solid black;
-- 
2.1.1

Bug#767134: ITP: ympd -- Standalone MPD web GUI

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: ympd
  Version: 1.2.3
  Upstream Author: Andy a...@ndyk.de
* URL: https://github.com/notandy/ympd
* License: GPL-2, Apache-2, MIT
  Programming Lang: C, HTML, CSS, JavaScript
  Description: Standalone MPD Web GUI

This program is a daemon that lets clients connect to it through the
HTTP protocol. It acts as a MPD (Music Player Daemon) client. It allows
one to browse and update the mpd database, manage playlists as well as
control various MPD settings, from a web browser.

The daemon (ympd) and the web browser can communicate together via
WebSockets, avoiding refreshing the page for each update (different song
playing, status change, playlist changes, etc...)

It uses Twitter Bootstrap as it's front-end framework, allowing one to
connect to it via from all sort of devices: desktop, mobile phones as
well as tablets.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#766172: chromium: dpkg errors on initial installation (--compare-versions takes three arguments: version relation version)

[Jason Pleau]

fixed 766172 38.0.2125.101-3
thanks


Hello !

I can reproduce the issue in 38.0.2125.101-2+b1 by purging and
re-installing chromium. However with version 38.0.2125.101-3 (from sid)
I am not getting this error anymore.


--

Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#765765: deluge: Deluge clients cannot connect to deluged because of SSL error

[Jason Pleau]

Hello !

The bug has been reported upstream already:
http://dev.deluge-torrent.org/ticket/2555

A patch was made to fix the issue (it works for me on 1.13.10)

I have attached the patch from the above url.

Cheers

-- 
Jason Pleau
diff --git a/deluge/core/rpcserver.py b/deluge/core/rpcserver.py
index 649c2ce..65601df 100644
--- a/deluge/core/rpcserver.py
+++ b/deluge/core/rpcserver.py
@@ -131,7 +131,8 @@ def getContext(self):
 SSL transport.
 
 ssl_dir = deluge.configmanager.get_config_dir(ssl)
-ctx = SSL.Context(SSL.SSLv3_METHOD)
+ctx = SSL.Context(SSL.SSLv23_METHOD)
+ctx.set_options(SSL.OP_NO_SSLv2  SSL.OP_NO_SSLv3)
 ctx.use_certificate_file(os.path.join(ssl_dir, daemon.cert))
 ctx.use_privatekey_file(os.path.join(ssl_dir, daemon.pkey))
 return ctx

Bug#748867: Missing API keys, some features of chromium are disabled

[Jason Pleau]

I just updated to 38.0.2125.101-1 in unstable and am now getting the
same warning when I start Chromium.

This is the first time I see it (upgraded from 37.0.2062.120-3)

-- 
Jason Pleau


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#762897: RFS: i3blocks/1.2-2 [ITP] - statusline for the i3 window manager

[Jason Pleau]

Hello again!

I have updated my i3blocks package to version 1.2-2 :

 * Package name: i3blocks
   Version : 1.2-2
   Upstream Author : Vivien Didelot viv...@didelot.org
 * URL : https://github.com/vivien/i3blocks
 * License : GPL-3
   Section : x11


It builds the following binary package:

  i3blocks   - highly flexible status line for the i3 window manager


To access further information about this package, please visit the
following URL:

  http://mentors.debian.net/package/i3blocks


Alternatively, one can download the package with dget using this command:

dget -x
http://mentors.debian.net/debian/pool/main/i/i3blocks/i3blocks_1.2-2.dsc

Change since the last upload :

  * Add a DEP-3 compliant header to 01-add-libexecdir-var-in-makefile.diff
  * Install the files inside the scripts/ folder in /usr/share as they
all are architecture-independent scripts (Debian Policy 9.1.1)
  * Bump Standards-Version to 3.9.6


Thank you!


-- 
Jason Pleau

PGP key: https://jpleau.ca/public_key.asc



signature.asc
Description: OpenPGP digital signature

Bug#762897: RFS: i3blocks/1.2-1 [ITP] - statusline for the i3 window manager

[Jason Pleau]

Package: sponsorship-requests
Severity: wishlist

Hello mentors!

I am looking for a sponsor for my package i3blocks


 * Package name: i3blocks
   Version : 1.2-1
   Upstream Author : Vivien Didelot viv...@didelot.org
 * URL : https://github.com/vivien/i3blocks
 * License : GPL-3
   Section : x11


It builds the following binary package:

  i3blocks   - highly flexible status line for the i3 window manager


To access further information about this package, please visit the
following URL:

  http://mentors.debian.net/package/i3blocks


Alternatively, one can download the package with dget using this command:

dget -x
http://mentors.debian.net/debian/pool/main/i/i3blocks/i3blocks_1.2-1.dsc

Change since the last upload :

i3blocks (1.2-1) unstable; urgency=low

  * Initial release (Closes: #762517)

 -- Jason Pleau ja...@jpleau.ca  Tue, 23 Sep 2014 21:30:32 -0400



A bit of information: The patch in my package that edits the Makefile
has been sent and accepted upstream.

An upcoming 1.3 release of the software is planned; my patches won't be
necessary when I update my debian package to 1.3.

As this is my first package that I upload, I am hoping to get criticisms
and suggestions, even if this is a really small package

Thanks !

Jason


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#762517: ITP: i3blocks - highly flexible status line for the i3 window manager

[Jason Pleau]

Package: wnpp
Severity: wishlist
Owner: Jason Pleau ja...@jpleau.ca

* Package name: i3blocks
  Version: 1.2
  Upstream Author: Vivien Didelot viv...@didelot.org
* URL : https://github.com/vivien/i3blocks
* License : GPL-3
  Programming Lang: C
  Description: highly flexible status line for the i3 window manager.

It follows the i3bar protocol; outputs JSON text which can be parsed 
with i3bar to display useful information in the status bar such as the 
current time, system load, uptime, available package updates, and much more.


A configuration file holds the settings for each block, which contains 
a script or a command that the user provides (upstream provides a few 
scripts). Blocks can be updated at a given internal, by a mouse 
click/wheel movement, or a signal.



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#730742: Font rendering fuzzy (straight lines smeared across subpixels) since upgrade to 2.5.1

[Jason Pleau]

I can see that this patch fixed the issue for some people, I (and maybe 
others too?) preferred the way the fonts were rendered with the Adobe 
hinter.


From what I can see we can't have both in the same package as the 
engine used (freetype or adobe) is decided at compile time.


Is there something we can do to allow both versions to be in Debian?

Perhaps another package with the Adobe engine enabled, that would 
conflict with libfreetype6 ?


Jason


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#756400: pytaglib: Please include a python2 package

[Jason Pleau]

Source: pytaglib
Severity: wishlist

Hello!

Would it be possible to include a python-pytaglib package from this source
package, in order to have a python2 version?

The reason I do need it is because python-nautilus from the archives runs
extensions as python2, and the extension I am writing for Nautilus uses
pytaglib.

I realize that we want to go forward with Python and prefer 3 over 2, but I
don't think there is any harm in supporting both in two different packages.

This could be a bug for nautilus-python instead, but I'm not sure if it's
possible to have it run extensions in both python2 and python3, or if it could
break peoples python extension. I filed it here, since this is (imho) the
easier way.

(Also, upstream seem to have fixed his non-free file, so if we do a version
bump (to 0.4.1) I *think* we could remove the dfsg level from the version.)

Thanks!





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

Kernel: Linux 3.15.6 (SMP w/8 CPU cores)
Locale: LANG=en_CA.UTF-8, LC_CTYPE=en_CA.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org