Re: Bug#883702: RFS: lina/5.3.0-1 ( #859130 ITP: lina -- iso-compliant Forth interpreter and compiler )

2018-07-09 Thread Albert van der Horst

Adam Borowski schreef op 2018-07-06 04:18:

On Mon, Jul 02, 2018 at 05:32:40PM +0200, Albert van der Horst wrote:

(An earlier non-personal call for reviews yielded no reactions.)
I eagerly await your review of
https://mentors.debian.net/package/lina
especially whether it complies with the "preferred form of 
modification"

guidelines.


The current version of the package fails to build:

makeinfo lina.texinfo --no-split -o lina.info
make[1]: makeinfo: Command not found
make[1]: *** [Makefile:52: lina.info] Error 127

which is trivially fixable by adding texinfo to Build-Depends.

I've fixed that now.




Besides that, in a cursory review, I don't see any other issues.

That is a relief.



I can't claim I understand your build system, though.

I can't see what is so difficult about it.

Let me translate the assembler to c.
I originally wanted to deliver the preprocessed file, but
that was not in correspondance with Debian policy.
The unprocessed file is called ci86.gnr and looks like
(in c-parlance).
--8<---
/*   
*/

#ifdef STANDALONE
   
#endif
...

#ifdef MS-WINDOWS
   /* OPEN-FILE */
   
#endif
 ...
#ifdef LINUX
   /* OPEN-FILE */
   
#endif
 ...
#ifdef MSDOS
   /* OPEN-FILE */
   
#endif

#ifdef FAS
  #if SIZE=32
 #define AX EAX
 etc.
  #endif
  #if SIZE=64
 #define AX RAX
 etc.
  #endif
#endif

#ifdef GAS
  #if SIZE=32
 #define AX %EAX
 etc.
  #endif
  #if SIZE=64
 #define AX %RAX
 etc.
  #endif
#endif
   ...
   /* code for DROP */
   

  ...
  ...
--
Some batches of constants are put together in an include file,
this is not shown because it would merely clutter up the example.
Now what happens is the file is preprocessed with LINUX defined,
FAS defined and SIZE equal to 32.
Of course the unix standard macro processor m4 is used,
not the c preprocessor.

The resulting assembler file is assembled by fasm, that has no
intervening object file and hence needs no linking step.
Hardly a "build system".



 And unlike not
understanding merely the packaged program's functionality, I don't 
think I
can sign this myself.  Thus, it'd be nice if someone else took a look 
as
well.  I don't quite see what's the new relation between ciforth and 
lina,

too.
I learned from Geert Stappers that I should not have asked you to review 
the

package, merely asked for comment, which you gave.
A verdict wether this package is suitable for Debian has to wait for 
someone

who has some affinity with it, I understand that.




Meow!

Thank you for your comment.

Groetjes Albert

--
Suffering is the prerogative of the strong, the weak -- perish.
Albert van der Horst



Bug#903433: marked as done (RFS: streamlink/0.14.2+dfsg-1~bpo9+1)

2018-07-09 Thread Debian Bug Tracking System
Your message dated Tue, 10 Jul 2018 01:13:02 +0200
with message-id <20180709231302.sqqqm4y7lmoyf...@angband.pl>
and subject line Re: Bug#903433: RFS: streamlink/0.14.2+dfsg-1~bpo9+1
has caused the Debian Bug report #903433,
regarding RFS: streamlink/0.14.2+dfsg-1~bpo9+1
to be marked as done.

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

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


-- 
903433: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=903433
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
Severity: wishlist
X-Debbugs-CC: debian-backpo...@lists.debian.org

Dear mentors and backports developers,

I am looking for a sponsor for my package "streamlink" into Debian
stretch-backports repository.

 * Package name: streamlink
   Version : 0.14.2+dfsg-1~bpo9+1
   Upstream Author : Streamlink Team
 * URL : https://streamlink.github.io/
 * License : BSD-2-clause, Apache-2.0, MIT/Expat, SIL-OFL-1.1
   Section : python

It builds those binary packages:

  livestreamer - transitional package - streamlink
  python3-streamlink - Python module for extracting video streams from
various websites
  python3-streamlink-doc - CLI for extracting video streams from various
websites (documenta
  streamlink - CLI for extracting video streams from various websites to
a video

To access further information about this package, please visit the
following URL:
  https://mentors.debian.net/package/streamlink


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

  dget -x
https://mentors.debian.net/debian/pool/main/s/streamlink/streamlink_0.14.2+dfsg-1~bpo9+1.dsc

More information about streamlink can be obtained from
https://streamlink.github.io/

Changes since the last backport upload (0.12.1+dfsg-1~bpo9+1):
streamlink (0.14.2+dfsg-1~bpo9+1) stretch-backports; urgency=medium

  * Rebuild for stretch-backports.
  * pybuild test: implement d/pybuild.testfiles in d/rules

 -- Alexis Murzeau   Mon, 09 Jul 2018 23:49:53 +0200

streamlink (0.14.2+dfsg-1) unstable; urgency=low

  * New upstream version 0.14.2+dfsg

 -- Alexis Murzeau   Sun, 01 Jul 2018 00:48:11 +0200

streamlink (0.14.0+dfsg-1) unstable; urgency=low

  * tests: fix shellcheck warnings
  * docs: move fonts-font-awesome and lib-modernizr to recommends
  * docs: revert to fonts-font-awesome v4
  * tests: check modernizr symlink in check_docs and refactor it
  * New upstream version 0.14.0+dfsg
  * Remove patches applied upstream: freezegun, missing test files,
escape sequences
  * Update patches

 -- Alexis Murzeau   Fri, 29 Jun 2018 00:16:46 +0200

streamlink (0.13.0+dfsg-1) unstable; urgency=low

  * Recommends all supported players (Closes: #898351)
  * Update upstream signing key with 0xE3DB9E282E390FA0
  * New upstream version 0.13.0+dfsg
  * Patches changes:
- Use font-awesome 5 shim
- Use modernizr.js symlink
- Fix tests executions by adding missing mpd files
- Avoid lambda with freezegun to support <= v0.3.9
- rtlxl plugin: use raw string to fix escape sequences
- Forwarded new patches to upstream
- Update patch remove_new_version_check
  * Add new build dependencies freezegun, recommonmark and isodate
  * Update autopkgtests for font files and js symlinks

 -- Alexis Murzeau   Tue, 12 Jun 2018 23:52:56 +0200


Differences from testing package (0.14.2+dfsg-1):
  * pybuild test: implement d/pybuild.testfiles in d/rules
  * Use debhelper 10 as 11 is not available.
  * pycountry: add patch to support stable provided version (1.8)
  * d/control: require pycountry <= 1.15


Regards,
-- 
Alexis Murzeau
PGP: B7E6 0EBB 9293 7B06 BDBC  2787 E7BD 1904 F480 937F



signature.asc
Description: OpenPGP digital signature
--- End Message ---
--- Begin Message ---
On Tue, Jul 10, 2018 at 12:11:43AM +0200, Alexis Murzeau wrote:
>  * Package name: streamlink
>Version : 0.14.2+dfsg-1~bpo9+1

> Changes since the last backport upload (0.12.1+dfsg-1~bpo9+1):
> streamlink (0.14.2+dfsg-1~bpo9+1) stretch-backports; urgency=medium
> 
>   * Rebuild for stretch-backports.
>   * pybuild test: implement d/pybuild.testfiles in d/rules
> 
>  -- Alexis Murzeau   Mon, 09 Jul 2018 23:49:53 +0200
[lots of unstable commits]

Looks good, uploaded.


Meow.
-- 
// If you believe in so-called "intellectual property", please immediately
// cease using counterfeit alphabets.  Instead, contact the nearest temple
// of Amon, whose priests will provide you with scribal services for all
// your writing needs, for Reasonable And Non-Discriminatory prices.--- End Message ---


Bug#903402: marked as done (RFS: fcitx-fbterm/0.2.0-3 [RC])

2018-07-09 Thread Debian Bug Tracking System
Your message dated Mon, 09 Jul 2018 22:20:18 +
with message-id 
and subject line closing RFS: fcitx-fbterm/0.2.0-3 [RC]
has caused the Debian Bug report #903402,
regarding RFS: fcitx-fbterm/0.2.0-3 [RC]
to be marked as done.

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

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


-- 
903402: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=903402
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
Severity: important
X-Debbugs-CC: debian-input-met...@lists.debian.org a...@debian.org 
sunwea...@debian.org s...@debian.org

Dear Mike, debian-input-method members and mentors,

I am looking for a sponsor for team package "fcitx-fbterm".

 * Package name: fcitx-fbterm
   Version : 0.2.0-3
   Upstream Author : Weng Xuetian 
 * URL : https://gitlab.com/fcitx/fcitx-fbterm
 * License : GPL-2+
   Section : utils

  It builds those binary packages:

fcitx-frontend-fbterm - Flexible Input Method Framework - FbTerm frontend

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

  https://mentors.debian.net/package/fcitx-fbterm


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

dget -x 
https://mentors.debian.net/debian/pool/main/f/fcitx-fbterm/fcitx-fbterm_0.2.0-3.dsc


  Git packaging repository:

https://salsa.debian.org/input-method-team/fcitx-fbterm.git


  Changes since the last upload:

 fcitx-fbterm (0.2.0-3) unstable; urgency=medium
 .
   * Team upload.
   * debian/control:
 - std-ver: 3.9.4 -> 4.1.5.
 - Bump debhelper compat to v11.
 - Use canonical Vcs URL on Salsa platform.
 - Use GitLab as homepage.
 - Use debian-input-method maillist in maintainer field.
   Closes: #899772.
 - Use @debian.org mail address for YunQiang Su in uploaders field.
   * Point d/watch at download.fcitx-im.org.
   * Apply "wrap-and-sort -abst".
   * debian/README.Debian: Rewrite document and usage.
   * d/rules:
 - Enable full hardening.
 - Use "dh_missing --fail-missing".
   * d/patches: Add a patch to point users to README.Debian file in
 "fcitx-fbterm-helper -h" output.

--
Regards,
Boyuan Yang

signature.asc
Description: This is a digitally signed message part.
--- End Message ---
--- Begin Message ---
Package fcitx-fbterm version 0.2.0-3 is in unstable now.
https://packages.qa.debian.org/fcitx-fbterm--- End Message ---


Bug#903433: RFS: streamlink/0.14.2+dfsg-1~bpo9+1

2018-07-09 Thread Alexis Murzeau
Package: sponsorship-requests
Severity: wishlist
X-Debbugs-CC: debian-backpo...@lists.debian.org

Dear mentors and backports developers,

I am looking for a sponsor for my package "streamlink" into Debian
stretch-backports repository.

 * Package name: streamlink
   Version : 0.14.2+dfsg-1~bpo9+1
   Upstream Author : Streamlink Team
 * URL : https://streamlink.github.io/
 * License : BSD-2-clause, Apache-2.0, MIT/Expat, SIL-OFL-1.1
   Section : python

It builds those binary packages:

  livestreamer - transitional package - streamlink
  python3-streamlink - Python module for extracting video streams from
various websites
  python3-streamlink-doc - CLI for extracting video streams from various
websites (documenta
  streamlink - CLI for extracting video streams from various websites to
a video

To access further information about this package, please visit the
following URL:
  https://mentors.debian.net/package/streamlink


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

  dget -x
https://mentors.debian.net/debian/pool/main/s/streamlink/streamlink_0.14.2+dfsg-1~bpo9+1.dsc

More information about streamlink can be obtained from
https://streamlink.github.io/

Changes since the last backport upload (0.12.1+dfsg-1~bpo9+1):
streamlink (0.14.2+dfsg-1~bpo9+1) stretch-backports; urgency=medium

  * Rebuild for stretch-backports.
  * pybuild test: implement d/pybuild.testfiles in d/rules

 -- Alexis Murzeau   Mon, 09 Jul 2018 23:49:53 +0200

streamlink (0.14.2+dfsg-1) unstable; urgency=low

  * New upstream version 0.14.2+dfsg

 -- Alexis Murzeau   Sun, 01 Jul 2018 00:48:11 +0200

streamlink (0.14.0+dfsg-1) unstable; urgency=low

  * tests: fix shellcheck warnings
  * docs: move fonts-font-awesome and lib-modernizr to recommends
  * docs: revert to fonts-font-awesome v4
  * tests: check modernizr symlink in check_docs and refactor it
  * New upstream version 0.14.0+dfsg
  * Remove patches applied upstream: freezegun, missing test files,
escape sequences
  * Update patches

 -- Alexis Murzeau   Fri, 29 Jun 2018 00:16:46 +0200

streamlink (0.13.0+dfsg-1) unstable; urgency=low

  * Recommends all supported players (Closes: #898351)
  * Update upstream signing key with 0xE3DB9E282E390FA0
  * New upstream version 0.13.0+dfsg
  * Patches changes:
- Use font-awesome 5 shim
- Use modernizr.js symlink
- Fix tests executions by adding missing mpd files
- Avoid lambda with freezegun to support <= v0.3.9
- rtlxl plugin: use raw string to fix escape sequences
- Forwarded new patches to upstream
- Update patch remove_new_version_check
  * Add new build dependencies freezegun, recommonmark and isodate
  * Update autopkgtests for font files and js symlinks

 -- Alexis Murzeau   Tue, 12 Jun 2018 23:52:56 +0200


Differences from testing package (0.14.2+dfsg-1):
  * pybuild test: implement d/pybuild.testfiles in d/rules
  * Use debhelper 10 as 11 is not available.
  * pycountry: add patch to support stable provided version (1.8)
  * d/control: require pycountry <= 1.15


Regards,
-- 
Alexis Murzeau
PGP: B7E6 0EBB 9293 7B06 BDBC  2787 E7BD 1904 F480 937F



signature.asc
Description: OpenPGP digital signature


Re: [Pkg-utopia-maintainers] Bug#895261: RFS: dbus-broker/13-2 [ITP] -- Linux D-Bus Message Broker

2018-07-09 Thread Felipe Sateler
On Sun, Jul 8, 2018 at 9:34 PM Michael Biebl  wrote:

> Hi Daniele
>
> Am 09.07.2018 um 01:45 schrieb Daniele Nicolodi:
> > my attempt to implement user service management in init-system-helpers
> > and debhelper unfortunately stalled by lack of review of the
> > init-system-helper patches.  Before I loose all interest, how do prefer
> > to solve the issue of user units in the dbus-broker package?
>
> I sincerely apologize for that.
>

Let me apologize too.


> I'm a bit tight on time atm, but it should get better in a few weeks.
> My perl is a bit rusty, but I'll try to have a look at it then. I hope
> you can bare with us for a little longer.
> Felipe, would you be able to review the patches?
>

Unfortunately I don't think so, at least not until August.

I did take a look, and it looked reasonable to me. I have not taken a look
at the latest iteration though. I was tempted to do a merge and release
(early and often, right?) but I'm going to be particularly unavailable for
about a month so I wouldn't be able to do any necessary followups in a
timely manner.

-- 

Saludos,
Felipe Sateler


Bug#903411: RFS: tvb-gdist/1.5.6-1 [ITP] -- friendly greeter

2018-07-09 Thread Umar Parooq
Package: sponsorship-requests
  Severity: normal [wishlist]

  Dear mentors,

  I am looking for a sponsor for my package "tvb-gdist"

 * Package name: tvb-gdist
   Version : 1.5.6-1
   Upstream Author : [TVB TEAM ]
 * URL : [https://github.com/the-virtual-brain/tvb-geodesic]
 * License : [GPL-3.0+]
   Section : python, c++

  It builds those binary packages:

tvb-gdist  - gdist module is a Cython interface

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

  https://mentors.debian.net/package/tvb-gdist


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

dget -x 
https://mentors.debian.net/debian/pool/main/t/tvb-gdist/tvb-gdist_1.5.6-1.dsc

  More information about hello can be obtained from https://www.example.com.

  Changes since the last upload:


  Regards,
   umar haruna abdullahi


Bug#903410: RFS: tvb-library/1.5.5.1-1 [ITP]

2018-07-09 Thread Umar Parooq
Package: sponsorship-requests
  Severity: normal

  Dear mentors,

  I am looking for a sponsor for my package "tvb-library"

 * Package name: tvb-library
   Version : 1.5.5.1-1
   Upstream Author : [TVB TEAM ]
 * URL : [https://www.thevirtualbrain.org]
 * License : [GPL-3.0+]
   Section : python

  It builds those binary packages:

tvb-library - most important scientific contribution of TVB Project

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

  https://mentors.debian.net/package/tvb-library


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

dget -x 
https://mentors.debian.net/debian/pool/main/t/tvb-library/tvb-library_1.5.5.1-1.dsc

  More information about tvb-library can be obtained from
https://github.com/the-virtual-brain/tvb-library.


  Regards,
   umar haruna abdullahi


Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Marc Haber
On Mon, Jul 09, 2018 at 09:12:05AM -0600, Daniele Nicolodi wrote:
> On 7/9/18 12:49 AM, Andrey Rahmatullin wrote:
> > On Sun, Jul 08, 2018 at 07:52:19PM -0600, Daniele Nicolodi wrote:
> >> The package builds just fine without intermediated cleaning, it is just
> >> gbp that complains about uncommitted changes to the source tree. That's
> >> why I think I'm missing something about the gbp workflow, as I think
> >> that systematically using --git-ignore-new is not the right thing.
> >
> > The gbp workflow implies you are not building in-tree.
> 
> Thanks! That is what I was missing.
> 
> Would it make sense to have this highlighted more prominently in the
> documentation of gbp? I don't recall seeing it mentioned anywhere. I can
> prepare a patch. Where would be a good place to add it?
> 
> Why is building off-tree the recommended workflow?

I have this exact problem as well. I solved it for me by using a
build-area setup as I am used to it back from svn-buildpackage.

Can I have some explanation, what in-free and off-tree building means
and how that compares to the build-area feature?

Greetings
Marc

-- 
-
Marc Haber | "I don't trust Computers. They | Mailadresse im Header
Leimen, Germany|  lose things."Winona Ryder | Fon: *49 6224 1600402
Nordisch by Nature |  How to make an American Quilt | Fax: *49 6224 1600421



Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Andrey Rahmatullin
On Mon, Jul 09, 2018 at 09:12:05AM -0600, Daniele Nicolodi wrote:
> >> The package builds just fine without intermediated cleaning, it is just
> >> gbp that complains about uncommitted changes to the source tree. That's
> >> why I think I'm missing something about the gbp workflow, as I think
> >> that systematically using --git-ignore-new is not the right thing.
> >
> > The gbp workflow implies you are not building in-tree.
> 
> Thanks! That is what I was missing.
> 
> Would it make sense to have this highlighted more prominently in the
> documentation of gbp? I don't recall seeing it mentioned anywhere. I can
> prepare a patch. Where would be a good place to add it?
> 
> Why is building off-tree the recommended workflow?
All I know is https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=859594#17


-- 
WBR, wRAR


signature.asc
Description: PGP signature


Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Daniele Nicolodi
On 7/8/18 8:21 PM, Tong Sun wrote:
> See the recommendation here
>  https://lists.debian.org/debian-mentors/2018/07/msg00086.html
> from Shengjing on using `gbp buildpackage export-dir`
> feature and see if it might help. It helped for my case. 

I believe that you are referring too the '--git-export-dir' option.

Cheers,
Dan



Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Daniele Nicolodi
On 7/9/18 12:49 AM, Andrey Rahmatullin wrote:
> On Sun, Jul 08, 2018 at 07:52:19PM -0600, Daniele Nicolodi wrote:
>> The package builds just fine without intermediated cleaning, it is just
>> gbp that complains about uncommitted changes to the source tree. That's
>> why I think I'm missing something about the gbp workflow, as I think
>> that systematically using --git-ignore-new is not the right thing.
>
> The gbp workflow implies you are not building in-tree.

Thanks! That is what I was missing.

Would it make sense to have this highlighted more prominently in the
documentation of gbp? I don't recall seeing it mentioned anywhere. I can
prepare a patch. Where would be a good place to add it?

Why is building off-tree the recommended workflow?

Cheers,
Dan



Bug#903402: RFS: fcitx-fbterm/0.2.0-3 [RC]

2018-07-09 Thread Boyuan Yang
Package: sponsorship-requests
Severity: important
X-Debbugs-CC: debian-input-met...@lists.debian.org a...@debian.org 
sunwea...@debian.org s...@debian.org

Dear Mike, debian-input-method members and mentors,

I am looking for a sponsor for team package "fcitx-fbterm".

 * Package name: fcitx-fbterm
   Version : 0.2.0-3
   Upstream Author : Weng Xuetian 
 * URL : https://gitlab.com/fcitx/fcitx-fbterm
 * License : GPL-2+
   Section : utils

  It builds those binary packages:

fcitx-frontend-fbterm - Flexible Input Method Framework - FbTerm frontend

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

  https://mentors.debian.net/package/fcitx-fbterm


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

dget -x 
https://mentors.debian.net/debian/pool/main/f/fcitx-fbterm/fcitx-fbterm_0.2.0-3.dsc


  Git packaging repository:

https://salsa.debian.org/input-method-team/fcitx-fbterm.git


  Changes since the last upload:

 fcitx-fbterm (0.2.0-3) unstable; urgency=medium
 .
   * Team upload.
   * debian/control:
 - std-ver: 3.9.4 -> 4.1.5.
 - Bump debhelper compat to v11.
 - Use canonical Vcs URL on Salsa platform.
 - Use GitLab as homepage.
 - Use debian-input-method maillist in maintainer field.
   Closes: #899772.
 - Use @debian.org mail address for YunQiang Su in uploaders field.
   * Point d/watch at download.fcitx-im.org.
   * Apply "wrap-and-sort -abst".
   * debian/README.Debian: Rewrite document and usage.
   * d/rules:
 - Enable full hardening.
 - Use "dh_missing --fail-missing".
   * d/patches: Add a patch to point users to README.Debian file in
 "fcitx-fbterm-helper -h" output.

--
Regards,
Boyuan Yang

signature.asc
Description: This is a digitally signed message part.


Bug#903387: RFS: fcitx-sunpinyin/0.4.2-2 [RC]

2018-07-09 Thread Osamu Aoki
Hi,

>  * Package name: fcitx-sunpinyin
>Version : 0.4.2-2
Thanks ... uploaded. 



Re: gbp upstream to pristine-tar, and import-orig filter

2018-07-09 Thread Andrey Rahmatullin
On Mon, Jul 09, 2018 at 09:25:47AM -0400, Tong Sun wrote:
> > > Did you create `pristine-tar` manually before calling `gbp buildpackage`?
> > This is wrong.
> 
> So would you explain what steps are correct instead then, to get
> upstream & pristine-tar branches from alien git from scratch, please?
I've already pointed you to the relevant gbp docs section. If you are
going to use some upstream branch as the gbp upstream-branch directly you
should just edit debian/gbp.conf. The pristine-tar branch should be
created automatically when pristine-tar is first used by gbp.

-- 
WBR, wRAR


signature.asc
Description: PGP signature


Re: gbp upstream to pristine-tar, and import-orig filter

2018-07-09 Thread Tong Sun
On Mon, Jul 9, 2018 at 2:53 AM Andrey Rahmatullin wrote:
>
> On Sun, Jul 08, 2018 at 06:16:47PM -0400, Tong Sun wrote:
> > Did you create `pristine-tar` manually before calling `gbp buildpackage`?
> This is wrong.

So would you explain what steps are correct instead then, to get
upstream & pristine-tar branches from alien git from scratch, please?



Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Henrique de Moraes Holschuh
On Sun, 08 Jul 2018, Daniele Nicolodi wrote:
> On 08/07/2018 19:28, Henrique de Moraes Holschuh wrote:
> > On Sun, 08 Jul 2018, Daniele Nicolodi wrote:
> >> After a successful package build done with 'gbp buildpackage' the
> >> package directory is left in a state which requires to run 'debuild --
> >> clean' before being able to build the package again.
> > 
> > Packages must be buildable several times in a row *without* the need to
> > *manually* call the clean target.
> > 
> > If it needs intermediate cleaning, arrange for it to detect and do so
> > automatically.
> 
> The package builds just fine without intermediated cleaning, it is just
> gbp that complains about uncommitted changes to the source tree. That's

I see.  Yeah, that's gbp trying too hard to protect you from yourself :(

You can, of course, configure it to never warn, but then you lose that
last line of defense.

-- 
  Henrique Holschuh



Re: C++ help needed for spaced: call of overloaded 'abs(uint32_t&)' is ambiguous

2018-07-09 Thread Fabian Klötzl
Hi,

An unsigned integer cannot be negative. Try deleting the violating line
patternset.cpp:188 or even the whole surrounding if statement.

Best,
Fabian

On 09.07.2018 13:11, Andreas Tille wrote:
> patternset.cpp:188:23: error: call of overloaded 'abs(uint32_t&)' is ambiguous
>size = std::abs(size);



Re: C++ help needed for spaced: call of overloaded 'abs(uint32_t&)' is ambiguous

2018-07-09 Thread Andreas Tscharner

On 09.07.2018 13:11, Andreas Tille wrote:

Hi,



Hello,

[snip]

patternset.cpp:188:23: error: call of overloaded 'abs(uint32_t&)' is ambiguous
size = std::abs(size);
^


There is no abs function which takes uint32_t as parameter, so it does 
not know which one it should use.


Probably
size = std::abs((int)size);
compiles (untested), but as Andrey already pointed out: as "size" is 
already unsigned, this code makes no sense.

Better ask upstream what s/he wanted to do here...

HTH and best regards
Andreas
--
  ("`-''-/").___..--''"`-._
   `o_ o  )   `-.  ( ).`-.__.`)
   (_Y_.)'  ._   )  `._ `. ``-..-'
 _..`--'_..-_/  /--'_.' .'
(il).-''  (li).'  ((!.-'

Andreas Tscharner   a...@stupidmail.ch   ICQ-No. 14356454



Re: C++ help needed for spaced: call of overloaded 'abs(uint32_t&)' is ambiguous

2018-07-09 Thread Andrey Rahmatullin
On Mon, Jul 09, 2018 at 01:11:34PM +0200, Andreas Tille wrote:
> patternset.cpp: In member function 'void patternset::CheckParams()':
> patternset.cpp:187:10: warning: comparison of unsigned expression < 0 is 
> always false [-Wtype-limits]
>   if(size < 0){
>  ~^~~
> patternset.cpp:188:23: error: call of overloaded 'abs(uint32_t&)' is ambiguous
>size = std::abs(size);
As size is unsigned, checking if it's < 0 and calling abs if it is is dead
code. No idea what did the author mean when writing this.

-- 
WBR, wRAR


signature.asc
Description: PGP signature


C++ help needed for spaced: call of overloaded 'abs(uint32_t&)' is ambiguous

2018-07-09 Thread Andreas Tille
Hi,

I take the freedom to ask for some C++ help for the new version of
spaced[1]:

...
g++ -DHAVE_CONFIG_H -I. -I..   -Wall -Wextra -std=c++11 -Wno-char-subscripts 
-Wdate-time -D_FORTIFY_SOURCE=2 -fopenmp -g -O2 
-fdebug-prefix-map=/build/spaced-1.2.0-201605+dfsg=. -fstack-protector-strong - 
  Wformat -Werror=format-security -c -o spaced-patternset.o `test -f 
'patternset.cpp' || echo './'`patternset.cpp
...
patternset.cpp: In member function 'void patternset::CheckParams()':
patternset.cpp:187:10: warning: comparison of unsigned expression < 0 is always 
false [-Wtype-limits]
  if(size < 0){
 ~^~~
patternset.cpp:188:23: error: call of overloaded 'abs(uint32_t&)' is ambiguous
   size = std::abs(size);
   ^
In file included from /usr/include/c++/7/cstdlib:77:0,
 from /usr/include/c++/7/bits/stl_algo.h:59,
 from /usr/include/c++/7/algorithm:62,
 from patternset.h:24,
 from patternset.cpp:21:
/usr/include/c++/7/bits/std_abs.h:78:3: note: candidate: constexpr long double 
std::abs(long double)
   abs(long double __x)
   ^~~
/usr/include/c++/7/bits/std_abs.h:74:3: note: candidate: constexpr float 
std::abs(float)
   abs(float __x)
   ^~~
/usr/include/c++/7/bits/std_abs.h:70:3: note: candidate: constexpr double 
std::abs(double)
   abs(double __x)
   ^~~
...


Any hint how to fix this?

Kind regards

   Andreas.


[1] https://salsa.debian.org/med-team/spaced

-- 
http://fam-tille.de



Bug#903387: RFS: fcitx-sunpinyin/0.4.2-2 [RC]

2018-07-09 Thread Boyuan Yang
Package: sponsorship-requests
Severity: important
X-Debbugs-CC: debian-input-met...@lists.debian.org sunwea...@debian.org 
a...@debian.org s...@debian.org

Dear Mike, debian-input-method members and mentors,

  I am looking for a sponsor for team package "fcitx-sunpinyin".

 * Package name: fcitx-sunpinyin
   Version : 0.4.2-2
   Upstream Author : Weng Xuetian 
 * URL : https://gitlab.com/fcitx/fcitx-sunpinyin/
 * License : GPL-3+
   Section : utils


  It builds those binary packages:

fcitx-sunpinyin - fcitx wrapper for Sunpinyin IM engine

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

  https://mentors.debian.net/package/fcitx-sunpinyin


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

dget -x 
https://mentors.debian.net/debian/pool/main/f/fcitx-sunpinyin/fcitx-sunpinyin_0.4.2-2.dsc


  Git packaging repository:

https://salsa.debian.org/input-method-team/fcitx-sunpinyin.git


  Changes since the last upload:

 fcitx-sunpinyin (0.4.2-2) unstable; urgency=medium
 .
   * Team upload.
   * debian/control:
 + Bump Standards-Version to 4.1.5. (no changes needed)
 + Bump debhelper compat to v11.
 + Use debian-input-method mailing list in maintainer field.
   Closes: #899506.
 + Use git repo on Salsa platform in Vcs fields.
 + Mark fcitx-sunpinnyin as M-A: same.
   * debian/rules: Use "dh_missing --fail-missing".
   * debian/copyright: Refresh information.
   * debian/changelog: Remove trailing spaces.


  Regards,
   Boyuan Yang

signature.asc
Description: This is a digitally signed message part.


Bug#902967: marked as done (RFS: ddnet/11.2.1-2)

2018-07-09 Thread Debian Bug Tracking System
Your message dated Mon, 9 Jul 2018 12:18:11 +0200
with message-id <20180709101811.jl3wtejifgd7o...@angband.pl>
and subject line Re: Bug#902967: RFS: ddnet/11.2.1-1
has caused the Debian Bug report #902967,
regarding RFS: ddnet/11.2.1-2
to be marked as done.

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

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


-- 
902967: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=902967
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "ddnet"

 * Package name: ddnet
   Version : 11.2.1-1
   Upstream Author : Dennis Felsing
 * URL : http://ddnet.tw/
 * License : zlib (code), CC-BY-SA 3.0 (data)
   Section : games

It builds those binary packages:

  ddnet - Teeworlds modification with a unique cooperative gameplay
 ddnet-data - Data for DDNet
 ddnet-server - Server for DDNet
 ddnet-tools - Tools for DDNet

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

https://mentors.debian.net/package/ddnet


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

  dget -x https://mentors.debian.net/debian/pool/main/d/ddnet/ddnet_11.2.1-2.dsc

More information about hello can be obtained from https://www.example.com.

Changes since the last upload:

  * New upstream release


Regards,
 Yangfl
--- End Message ---
--- Begin Message ---
On Wed, Jul 04, 2018 at 12:23:25PM +0800, Yangfl wrote:
>  * Package name: ddnet
>Version : 11.2.1-1

> Changes since the last upload:
> 
>   * New upstream release

✓

-- 
// If you believe in so-called "intellectual property", please immediately
// cease using counterfeit alphabets.  Instead, contact the nearest temple
// of Amon, whose priests will provide you with scribal services for all
// your writing needs, for Reasonable And Non-Discriminatory prices.--- End Message ---


Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Andrey Rahmatullin
On Mon, Jul 09, 2018 at 10:51:10AM +0200, Ferenc Wágner wrote:
> > gbp runs clean before the second build
> 
> Gbp runs clean before each build, but the default clean command was
> changed to /bin/true in version 0.6.9.
Ah indeed. 
Because of my first point.

-- 
WBR, wRAR


signature.asc
Description: PGP signature


Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Ferenc Wágner
Andrey Rahmatullin  writes:

> gbp runs clean before the second build

Gbp runs clean before each build, but the default clean command was
changed to /bin/true in version 0.6.9.  You can try

gbp buildpackage --git-cleaner="debuild -- clean"

or something similar.
-- 
Feri



Re: gbp upstream to pristine-tar, and import-orig filter

2018-07-09 Thread Andrey Rahmatullin
On Sun, Jul 08, 2018 at 06:16:47PM -0400, Tong Sun wrote:
> Did you create `pristine-tar` manually before calling `gbp buildpackage`?
This is wrong.

> I.e., besides the shc_3.9.6.orig.tar.gz.delta & shc_3.9.6.orig.tar.gz.id
> files, all files from my master are in there as well.
Sure, you created it from a branch with those files, why would they
disappear?

> I have tried to remove them all, but found that files in my master
> branch get deleted too. 
I don't think so.

-- 
WBR, wRAR


signature.asc
Description: PGP signature


Re: gbp clone, upstream-branch is not 'upstream' but `release`

2018-07-09 Thread Andrey Rahmatullin
On Sun, Jul 08, 2018 at 06:40:58PM -0400, Tong Sun wrote:
> gbp clone -v https://github.com/neurobin/shc.git
gbp clone is for gbp repos.

> from scratch, but found that my branches are:
> 
> $ git branch -v
> * master  556fd7a release after fix #38
>   release 379c501 release after fix #38
> 
> See? the upstream-branch is not 'upstream' but called `release` instead.
> Why?
It's not an upstream-branch. It's just a branch that exists at the remote
repo.

-- 
WBR, wRAR


signature.asc
Description: PGP signature


Re: Cleaning up after 'gbp buildpackage'

2018-07-09 Thread Andrey Rahmatullin
On Sun, Jul 08, 2018 at 07:52:19PM -0600, Daniele Nicolodi wrote:
> The package builds just fine without intermediated cleaning, it is just
> gbp that complains about uncommitted changes to the source tree. That's
> why I think I'm missing something about the gbp workflow, as I think
> that systematically using --git-ignore-new is not the right thing.
The gbp workflow implies you are not building in-tree.
Even if you are, gbp runs clean before the second build, so if there are
uncommitted changes then something is broken. Can you please provide more
specific info?

-- 
WBR, wRAR


signature.asc
Description: PGP signature