fonts-vazirmatn

[A. Shafiei]

Hello Steve,

Thanks for sponsoring the package.
I also fixed the warning in debian/watch and changed the target to kinetic
We have some other free farsi fonts that I will package and ask next time
for sponsorship.

Greetings.

On Sat, Sep 3, 2022 at 12:41 AM Steve Langasek <steve.langasek at ubuntu.com>
wrote:

> On Fri, Sep 02, 2022 at 11:48:03PM +0430, A. Shafiei wrote:
> > Hello Steve,
> >
> > Thanks for the feedback. All issues you mentioned are addressed and all
> are
> > fixed.
> > I added the debian/watch. I get a warning and I do not know why.
>
> The warning is because your download URL is http, which is insecure against
> man-in-the-middle attacks.  Running 'uscan' gives output showing you are
> redirected to https, so it seems using an https url is possible.  But this
> is only a warning and not a blocker for inclusion in Ubuntu.
>
> > I also changed the upstream-contact in copyright to the author of the
> font.
> > Could you please review the package?
> > https://code.launchpad.net/~farsi-fonts/+junk/fonts-vazirmatn
> > <
> https://deref-gmx.com/mail/client/RRy7eofpEnU/dereferrer/?redirectUrl=https%3A%2F%2Fcode.launchpad.net%2F%7Efarsi-fonts%2F%2Bjunk%2Ffonts-vazirmatn
> >
>
> > Sure, I will be happy if you could sponsor this package.
>
> > Thanks in advance.
>
> Thanks, I have sponsored the package now and it has been accepted into
> kinetic-proposed.
>
> Note that debian/changelog of your bzr branch showed it was targeting focal
> with the upload, so I had to change this.
>
> > On Fri, Sep 2, 2022 at 8:27 PM Steve Langasek <steve.langasek at ubuntu.com
> >
> > wrote:
> >
> > > Hi Shafiei,
> > >
> > > Some feedback:
> > >
> > > - please create a debian/watch file that says where to get (updates
> to) the
> > >   upstream source.  I was able to work out where to find the upstream
> > >   tarball by looking at debian/copyright, but ideally this would be
> much
> > >   more automatic.  (debian/watch file syntax is documented in the
> uscan(1)
> > >   manpage, but it is non-trivial;
> > >   https://lists.debian.org/debian-mentors/2022/04/msg00052.html might
> be a
> > >   useful starting point.)
> > > - "Description" in debian control is malformatted.  The description
> field
> > >   consists of a single-line short description (should fit under 72
> > >   characters) followed by an indented paragraph for the long
> description.
> > >   The way you have this formatted currently, is a 700-character short
> > >   description which will break display.  (Running the lintian command
> over
> > >   your built package would catch this error.)
> > > - you have the package marked as Architecture: any, but this means the
> > >   package is built separately for each architecture - you do not have
> any
> > >   architecture-specific contents here so the package should be
> > >   Architecture: all
> > > - debian/copyright declares
> > >   https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
> but
> > >   does not follow the format.  Your License: field should consist of a
> > >   single one-word name for the license, followed by indented
> paragraphs.
> > >   Continuation across multiple paragraphs should be encoded as ' .',
> not as
> > >   a blank line.  According to https://spdx.org/licenses/ the preferred
> > > short
> > >   name for the license used here is OFL-1.1.
> > > - you list yourself as the upstream contact in debian/copyright but it
> is
> > >   not clear to me why.  You are not listed in the upstream AUTHORS.txt
> and
> > >   https://gitlab.flossir.org/rshf who maintains the git repository is
> not
> > >   obviously you.
> > > - font packages should be declared 'Multi-Arch: foreign' for maximum
> > >   compatibility.  (This issue was also identified by lintian.)
> > >
> > > If you address the above issues in the packaging, I am happy to sponsor
> > > this
> > > package into the Ubuntu archive for the upcoming kinetic release.
> > >
> > >
> > >
> > >
> > > On Fri, Aug 26, 2022 at 10:56:10AM +0430, A. Shafiei wrote:
> > > > Hello everyone,
> > > >
> > > > There are a number of fonts for Persian language which I am trying to
> > > > package. The first one is the Vazirmatn font.
> > > >
> > > > I would like to know if anyone could please review this package:
> > > >
> > > > https://code.launchpad.net/~farsi-fonts/+junk/fonts-vazirmatn
> > > >
> > > >
> https://launchpad.net/~farsi-fonts/+archive/ubuntu/farsi-fonts/+packages
> > > >
> > > > Thanks in advance.
> > > > Shafiei
> > >
> > > > --
> > > > Ubuntu-motu mailing list
> > > > Ubuntu-motu at lists.ubuntu.com
> > > > Modify settings or unsubscribe at:
> > > https://lists.ubuntu.com/mailman/listinfo/ubuntu-motu
> > >
> > >
> > > --
> > > Steve Langasek                   Give me a lever long enough and a
> Free OS
> > > Debian Developer                   to set it on, and I can move the
> world.
> > > Ubuntu Developer
> https://www.debian.org/
> > > slangasek at ubuntu.com
> vorlon at debian.org
> > >
>
> --
> Steve Langasek                   Give me a lever long enough and a Free OS
> Debian Developer                   to set it on, and I can move the world.
> Ubuntu Developer                                   https://www.debian.org/
> slangasek at ubuntu.com                                     vorlon at debian.org
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/ubuntu-motu/attachments/20220903/307deb22/attachment-0001.html>