Bug#895262: RFS: deepin-menu/3.3.2-1

2018-04-11 Thread Yanhao Mo
On Tue 04/10 23:19, Adam Borowski wrote:
> 
> Hi!
> I see that the packaging no longer installs autostart parts:
> 
> --- deepin-menu-3.2.0/debian/install2017-08-11 15:49:21.0 +0200
> +++ deepin-menu-3.3.2/debian/install1970-01-01 01:00:00.0 +0100
> @@ -1,2 +0,0 @@
> -data/com.deepin.menu.service /usr/share/dbus-1/services/
> -deepin-menu.desktop /etc/xdg/autostart/
> 
> Files in first .deb but not in second
> -
> -rw-r--r--  root/root   /etc/xdg/autostart/deepin-menu.desktop
> -rw-r--r--  root/root   DEBIAN/conffiles
> 
> It's a non-obvious packaging change that would be nice to be described in
> the changelog.
> 
> On the other hand, DDE wasn't a part of Stretch (so not many users yet), and
> for your internal purposes, it's in git:
> https://github.com/linuxdeepin/deepin-menu/commit/09e65bd7bd46c9aa6cdd1b205aa4e4f7be33ef0c
> Speaking about nitpicks in the changelog: s/Standers/Standards/.
> 
> Thanks for your work!
> 

Hi Adam,
Thanks for pointing that and give such detailed information. I have
fixed that two issues and re-uploaded.

> so if you feel like I'm nitpicking too much, please tell me so, I can upload
> as-is.
Ok, I'll let you know if I have that feeling, haven't at least for now ◕‿◕.

-- 
Yanhao Mo


signature.asc
Description: PGP signature


Bug#895262: RFS: deepin-menu/3.3.2-1

2018-04-10 Thread Adam Borowski
On Mon, Apr 09, 2018 at 10:31:30AM +0800, Yanhao Mo wrote:
> * Package name: deepin-menu
>   Version : 3.3.2-1

(Note: there's a couple transitions going on, so installability of build
dependencies comes and goes, causing temporary problems; this is not your
fault.)

> Changes since the last upload:
>   * New upstream release.
>   * d/control: add new uploader Yanhao Mo 
>   * d/control: Use Salsa platform for Vcs field.
>   * debian/control: Update Standers-Version to 4.1.4.
>   * debian/compat: Switch compat to level 11.

Hi!
I see that the packaging no longer installs autostart parts:

--- deepin-menu-3.2.0/debian/install2017-08-11 15:49:21.0 +0200
+++ deepin-menu-3.3.2/debian/install1970-01-01 01:00:00.0 +0100
@@ -1,2 +0,0 @@
-data/com.deepin.menu.service /usr/share/dbus-1/services/
-deepin-menu.desktop /etc/xdg/autostart/

Files in first .deb but not in second
-
-rw-r--r--  root/root   /etc/xdg/autostart/deepin-menu.desktop
-rw-r--r--  root/root   DEBIAN/conffiles

It's a non-obvious packaging change that would be nice to be described in
the changelog.

On the other hand, DDE wasn't a part of Stretch (so not many users yet), and
for your internal purposes, it's in git:
https://github.com/linuxdeepin/deepin-menu/commit/09e65bd7bd46c9aa6cdd1b205aa4e4f7be33ef0c
so if you feel like I'm nitpicking too much, please tell me so, I can upload
as-is.


Speaking about nitpicks in the changelog: s/Standers/Standards/.

Thanks for your work!


喵!
-- 
⢀⣴⠾⠻⢶⣦⠀ 
⣾⠁⢰⠒⠀⣿⡁ 
⢿⡄⠘⠷⠚⠋⠀ ... what's the frequency of that 5V DC?
⠈⠳⣄



Bug#895262: RFS: deepin-menu/3.3.2-1

2018-04-08 Thread Yanhao Mo
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "deepin-menu"

* Package name: deepin-menu
  Version : 3.3.2-1
  Upstream Author : Deepin Technology Co., Ltd.
* URL : https://github.com/linuxdeepin/deepin-menu
* License : GPL-3+
  Section : x11

It builds those binary packages:

  deepin-menu - Deepin menu service

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

  https://mentors.debian.net/package/deepin-menu

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

  dget -x 
https://mentors.debian.net/debian/pool/main/d/deepin-menu/deepin-menu_3.3.2-1.dsc

More information about hello can be obtained from 
https://salsa.debian.org/pkg-deepin-team/deepin-menu

Changes since the last upload:
  * New upstream release.
  * d/control: add new uploader Yanhao Mo 
  * d/control: Use Salsa platform for Vcs field.
  * debian/control: Update Standers-Version to 4.1.4.
  * debian/compat: Switch compat to level 11.

-- 
Yanhao Mo


signature.asc
Description: PGP signature