** Branch linked: lp:~ubuntu-branches/ubuntu/saucy/youker-assistant
/saucy-proposed
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1213998
Title:
[needs-packaging] youker-assistant
To manage
** Changed in: ubuntukylin
Status: In Progress = Fix Released
** Changed in: ubuntu
Status: In Progress = Fix Released
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1213998
Title:
** Changed in: ubuntu
Status: New = In Progress
** Changed in: ubuntukylin
Status: New = Fix Committed
** Changed in: ubuntukylin
Status: Fix Committed = In Progress
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to
** Branch linked: lp:youker-assistant/0.1
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1213998
Title:
[needs-packaging] youker-assistant
To manage notifications about this bug go to:
New tarball attached. Fix lintian errors and improve descriptions.
Thanks to dholbach and happyaron.
** Attachment added: youker-assistant _0.1.5.tar.gz
https://bugs.launchpad.net/ubuntukylin/+bug/1213998/+attachment/3786594/+files/youker-assistant%20_0.1.5.tar.gz
** Description changed:
Here another very quick review until somebody in -desktop can have another look:
- the tarball contains .bzr - you might want to remove it from there
- http://paste.ubuntu.com/6016797/ has uncovered a few more issues, of which
- desktop-entry-lacks-keywords-entry might be interesting to fix
Things to be added (still before trying build the package):
1. The short and long descriptions in debian/control isn't good enough
yet, though it's much better than only having one sentence. An example:
Description: integrated tool to aid in routine system maintenance tasks for
desktop users
** Description changed:
'Youker Assistant' is a system assistant tool, and it was written using
Qt 4.8, QML and Python. It aimed at helping all users to manage their
UbuntuKylin conveniently. It will be installed in UbuntuKylin 13.10, a
Chinese flavor of Ubuntu.
Project URL:
** Description changed:
'Youker Assistant' is a system assistant tool, and it was written using
Qt 4.8, QML and Python. It aimed at helping all users to manage their
UbuntuKylin conveniently. It will be installed in UbuntuKylin 13.10, a
Chinese flavor of Ubuntu.
Project URL:
A few things I noticed:
- The package fails to build on amd64 because qmake is specified as
/usr/lib/i386-linux-gnu/qt4/bin/qmake in Makefile.
- debian/copyright does not list other copyright holders.
- Is libphonon-dev required as a dependency?
--
You received this bug notification because
New tarball attached, according to dholbach's comments. Thanks.
** Attachment added: youker-assistant_0.1.5.tar.gz
https://bugs.launchpad.net/ubuntukylin/+bug/1213998/+attachment/3779357/+files/youker-assistant_0.1.5.tar.gz
--
You received this bug notification because you are a member of
libphonon (in depends) does not exist as a package with that name - it
looks like shlibs:Depends already contains libphonon4 after the build.
I'll leave the rest of the review to the Desktop team.
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed
Yes, there is an error in depends. It should be libphonon4, and it needs
phonon-backend-gstreamer to work properly. we had modified it.
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1213998
Title:
new tarball Attached. Hi developers, any more comments on this project?
** Attachment added: youker-assistant _0.1.5.tar.gz
https://bugs.launchpad.net/ubuntukylin/+bug/1213998/+attachment/3780071/+files/youker-assistant%20_0.1.5.tar.gz
--
You received this bug notification because you are a
Attached is source code.
** Attachment added: youker-assistant_0.1.5.tar.gz
https://bugs.launchpad.net/ubuntu/+bug/1213998/+attachment/3778102/+files/youker-assistant_0.1.5.tar.gz
** Tags added: needs-packaging
** Also affects: ubuntukylin
Importance: Undecided
Status: New
**
** Description changed:
'Youker Assistant' is a system assistant tool, and it was written using
Qt 4.8, QML and Python. It aimed at helping all users to manage their
UbuntuKylin conveniently. It will be installed in UbuntuKylin 13.10, a
Chinese flavor of Ubuntu.
- URL:
I did a quick look at the branch without trying to build it, and here are my
questions:
1. debian/compat is 8, which should bump to 9 with no other change needed.
2. debian/control says section is gnome, while the application itself has
nothing to do with gnome, it's should be changed to
*** This is an automated message ***
This bug is tagged needs-packaging which identifies it as a request for
a new package in Ubuntu. As a part of the managing needs-packaging bug
reports specification,
https://wiki.ubuntu.com/QATeam/Specs/NeedsPackagingBugs, all needs-
packaging bug reports
Thanks. Attached is the new tarball, merged changes from Robert Bruce
Park and comments from Aron Xu.
** Attachment added: youker-assistant_0.1.5.tar.gz
https://bugs.launchpad.net/ubuntukylin/+bug/1213998/+attachment/3778775/+files/youker-assistant_0.1.5.tar.gz
--
You received this bug
19 matches
Mail list logo