Bug#883379: RFS: deepin-voice-recorder/1.3.6-1 [ITP]

[Yangfl]

control: tags -1 - moreinfo

After discussing with the upstream, they decide to refactor their
manual system and before that they agree to remove it for the time
being. (and patch 0001 is now useless)

Uploaded.

Bug#883379: RFS: deepin-voice-recorder/1.3.6-1 [ITP]

[Gianfranco Costamagna]

control: tags -1 moreinfo

> Is there any possibility of renaming this directory to deepin-manual
> or deepin-manuals (if deepin-manual is already taken by the
> deepin-manual package)? I think dman is too generic and likely to be
> confused for something related to man-db.
I agree here

G.



signature.asc
Description: OpenPGP digital signature

Bug#883379: RFS: deepin-voice-recorder/1.3.6-1 [ITP]

[Paul Wise]

On Tue, Mar 6, 2018 at 11:08 PM, Boyuan Yang wrote:

> As previously explained by Yangfl, files in /usr/share/dman/ are
> richtext manuals intended to be read by deepin-manual (which is not in
> Debian Archive yet).

Is there any possibility of renaming this directory to deepin-manual
or deepin-manuals (if deepin-manual is already taken by the
deepin-manual package)? I think dman is too generic and likely to be
confused for something related to man-db.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise

Bug#883379: RFS: deepin-voice-recorder/1.3.6-1 [ITP]

[Boyuan Yang]

Hi Yangfl, Gianfranco,

As previously explained by Yangfl, files in /usr/share/dman/ are
richtext manuals intended to be read by deepin-manual (which is not in
Debian Archive yet).

"Dman" manual files contain some images, which made lintian aware that
this is an arch-dep package with large arch-indep files.

Splitting the package into "deepin-voice-recorder" and
"deepin-voice-recorder-doc" might be an option. We could choose to
remove all files under /usr/share/dman/ too. A third option is to keep
everything intact since they do not harm at all. Personally I think
all three approaches are acceptable.

--
Regards,
Boyuan Yang

Bug#883379: RFS: deepin-voice-recorder/1.3.6-1 [ITP]

[Yangfl]

control: tags -1 - moreinfo

On Thu, 1 Mar 2018 17:35:11 +0100 Gianfranco Costamagna
 wrote:
> control: owner -1 !
> control: tags -1 moreinfo
>
> grep DOCDIR . -R
> ./deepin-voice-recorder.pro:isEmpty(DOCDIR):DOCDIR=/usr/share/dman/deepin-voice-recorder
> ./deepin-voice-recorder.pro:manual.path = $$INSTROOT$$DOCDIR
>
>
> Why is the package installing stuff in /usr/share/dman? nothing in the 
> archive uses such directory.
>

It's deepin manual system, and these files will be used by
https://github.com/linuxdeepin/deepin-manual to display a rich-text
help page.

> I'm reluctant to sponsor it, unless you provide some good explanation, and 
> post a patch to upstream.
> (BTW the patch 0001 looks useless, please forward upstream).
>
> The other std-version, debhelper bump, and so on still apply
> also lintian
> I: deepin-voice-recorder: desktop-entry-lacks-keywords-entry 
> usr/share/applications/deepin-voice-recorder.desktop

The 0001 and desktop-entry-lacks-keywords-entry have been reported to
upstream. I'd like to leave them as-is for the time being and wait for
next version fixing them. std-v, debhelper are bumped.

Uploaded to mentors.d.o

> Other stuff LGTM.
>

Bug#883379: RFS: deepin-voice-recorder/1.3.6-1 [ITP]

[Gianfranco Costamagna]

control: owner -1 !
control: tags -1 moreinfo

grep DOCDIR . -R
./deepin-voice-recorder.pro:isEmpty(DOCDIR):DOCDIR=/usr/share/dman/deepin-voice-recorder
./deepin-voice-recorder.pro:manual.path = $$INSTROOT$$DOCDIR


Why is the package installing stuff in /usr/share/dman? nothing in the archive 
uses such directory.

I'm reluctant to sponsor it, unless you provide some good explanation, and post 
a patch to upstream.
(BTW the patch 0001 looks useless, please forward upstream).

The other std-version, debhelper bump, and so on still apply
also lintian
I: deepin-voice-recorder: desktop-entry-lacks-keywords-entry 
usr/share/applications/deepin-voice-recorder.desktop


Other stuff LGTM.



signature.asc
Description: OpenPGP digital signature

Bug#883379: RFS: deepin-voice-recorder/1.3.6-1 [ITP]

[Yangfl]

Package: sponsorship-requests
Severity: wishlist
X-Debbugs-CC: pkg-deepin-de...@lists.alioth.debian.org

Dear mentors,

I am looking for a sponsor for my package "deepin-voice-recorder"

 * Package name: deepin-voice-recorder
   Version : 1.3.6-1
   Upstream Author : Deepin Technology Co., Ltd.
 * URL : https://github.com/linuxdeepin/deepin-voice-recorder
 * License : GPL-3+
   Section : utils


  It builds those binary packages:

deepin-voice-recorder - Voice recorder for Deepin Desktop Environment

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

  https://mentors.debian.net/package/deepin-voice-recorder

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

dget -x 
https://mentors.debian.net/debian/pool/main/d/deepin-voice-recorder/deepin-voice-recorder_1.3.6-1.dsc

  Git repo:

https://anonscm.debian.org/git/pkg-deepin/deepin-voice-recorder.git

Regards,
Yangfl