Bug#910837: RFS: mle/1.2-1 [ITP] -- flexible terminal-based text editor (C)

2018-12-22 Thread Mo Zhou
See comments I left in github issues.

On Fri, Dec 21, 2018 at 03:34:40PM -0500, a...@php.net wrote:
> On Wed, 17 Oct 2018 13:28:40 + Mo Zhou  wrote:
> > ...
> > Try to build against this termbox package, it's almost finished:
> > https://salsa.debian.org/debian/termbox
> >
> 
> Updated to build against uthash-dev, liblua5.3-dev, and libtermbox-dev. 
> (Thanks
> lumin!)
> 
> https://mentors.debian.net/package/mle
> 
> Adam



Bug#910837: RFS: mle/1.2-1 [ITP] -- flexible terminal-based text editor (C)

2018-12-21 Thread as
On Wed, 17 Oct 2018 13:28:40 + Mo Zhou  wrote:
> ...
> Try to build against this termbox package, it's almost finished:
> https://salsa.debian.org/debian/termbox
>

Updated to build against uthash-dev, liblua5.3-dev, and libtermbox-dev.
(Thanks lumin!)

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

Adam


Bug#910837: RFS: mle/1.2-1 [ITP] -- flexible terminal-based text editor (C)

2018-10-17 Thread Mo Zhou
On Sat, Oct 13, 2018 at 06:03:48PM -0400, a...@php.net wrote:
> Yes, I can make Lua and uthash package deps instead of embedding them.
> Is it ok to embed termbox for now, or should we consider that a
> blocker?

Try to build against this termbox package, it's almost finished:
https://salsa.debian.org/debian/termbox



Bug#910837: RFS: mle/1.2-1 [ITP] -- flexible terminal-based text editor (C)

2018-10-14 Thread Mo Zhou
On Sat, Oct 13, 2018 at 06:03:48PM -0400, a...@php.net wrote:
> Hi Mo,
> 
> Thanks for the info.
> 
> > 1. Can we avoid shipping with embedded code copy[1] of lua, termbox and
> >uthash? . lua 5.3 and uthash are already in the archive. termbox needs
> >to be packaged.
> 
> Yes, I can make Lua and uthash package deps instead of embedding them.
> Is it ok to embed termbox for now, or should we consider that a
> blocker?

I'm not sure if we have enough reason to ship with an embedded copy of
termbox, while at the same time make ftp-master happy.
Termbox is a library for general purpose instead of specific to mle.

My recommendation is to package termbox as an individual package and
block mle by that ITP.

Feel free to ask the mentors list if you encountered any problem.
 
> There is a packaging branch in the main repo by the way:
> https://github.com/adsr/mle/tree/debian

Oops. Thanks for the hint. Then you should put it to the
Vcs-Git and Vcs-Browser fields in debian/control.

> Thanks for the tip about the "Recommends" field. I will add optional
> runtime deps there.
> 
> Adam
> 
> On Fri, 12 Oct 2018 04:50:12 + Mo Zhou  wrote:
> > Hi Adam,
> >
> > Thanks for this package, the copyright file looks good to me.
> > However there are still a couple of problems:
> >
> > 1. Can we avoid shipping with embedded code copy[1] of lua, termbox and
> >uthash? . lua 5.3 and uthash are already in the archive. termbox needs
> >to be packaged.
> >
> > 2. I'd recommend you to create a packaging repo so that the others can
> >track the changes about packaging with git. Since you are also the
> >upstream author, it's fine if you set the source format as
> >"3.0 (native)" and push the debian directory to the upstream git repo.
> >A separate git repo following e.g. the gbp-buildpackage workflow is
> >also acceptable. No packaging repo is fine too but not encouraged.
> >
> > 3. External tools such as fzf can be leveraged by mle to enhance
> >features. That means those packages should be listed in the
> >"Recommends" or "Suggests" fields.
> >
> > Best.
> >
> > [1] https://wiki.debian.org/EmbeddedCodeCopies
> >
> > On Thu, Oct 11, 2018 at 08:04:33PM -0400, Adam Saponara wrote:
> > > Package: sponsorship-requests
> > > Severity: wishlist
> > >
> > > Dear mentors,
> > >
> > > I am looking for a sponsor for my package "mle":
> > >
> > >   Package name: mle
> > >   Version : 1.2-1
> > >   Upstream Author : Adam Saponara 
> > >   URL : https://github.com/adsr/mle
> > >   License : Apache, Expat, BSD-1-clause
> > >   Section : editors
> > >
> > > It builds those binary packages:
> > >
> > >   mle - flexible terminal-based text editor (C)
> > >
> > > To access further information about this package, please visit the 
> > > following URL:
> > >
> > >   https://mentors.debian.net/package/mle
> > >
> > > Alternatively, one can download the package with dget using this command:
> > >
> > >   dget -x https://mentors.debian.net/debian/pool/main/m/mle/mle_1.2-1.dsc
> > >
> > > More information about hello can be obtained from 
> > > https://github.com/adsr/mle.
> > >
> > > Changes since the last upload: -
> > >
> > > Regards,
> > >
> > > Adam
> > >
> >
> 



Bug#910837: RFS: mle/1.2-1 [ITP] -- flexible terminal-based text editor (C)

2018-10-13 Thread as
Hi Mo,

Thanks for the info.

> 1. Can we avoid shipping with embedded code copy[1] of lua, termbox and
>uthash? . lua 5.3 and uthash are already in the archive. termbox needs
>to be packaged.

Yes, I can make Lua and uthash package deps instead of embedding them.
Is it ok to embed termbox for now, or should we consider that a
blocker?

There is a packaging branch in the main repo by the way:
https://github.com/adsr/mle/tree/debian

Thanks for the tip about the "Recommends" field. I will add optional
runtime deps there.

Adam

On Fri, 12 Oct 2018 04:50:12 + Mo Zhou  wrote:
> Hi Adam,
>
> Thanks for this package, the copyright file looks good to me.
> However there are still a couple of problems:
>
> 1. Can we avoid shipping with embedded code copy[1] of lua, termbox and
>uthash? . lua 5.3 and uthash are already in the archive. termbox needs
>to be packaged.
>
> 2. I'd recommend you to create a packaging repo so that the others can
>track the changes about packaging with git. Since you are also the
>upstream author, it's fine if you set the source format as
>"3.0 (native)" and push the debian directory to the upstream git repo.
>A separate git repo following e.g. the gbp-buildpackage workflow is
>also acceptable. No packaging repo is fine too but not encouraged.
>
> 3. External tools such as fzf can be leveraged by mle to enhance
>features. That means those packages should be listed in the
>"Recommends" or "Suggests" fields.
>
> Best.
>
> [1] https://wiki.debian.org/EmbeddedCodeCopies
>
> On Thu, Oct 11, 2018 at 08:04:33PM -0400, Adam Saponara wrote:
> > Package: sponsorship-requests
> > Severity: wishlist
> >
> > Dear mentors,
> >
> > I am looking for a sponsor for my package "mle":
> >
> >   Package name: mle
> >   Version : 1.2-1
> >   Upstream Author : Adam Saponara 
> >   URL : https://github.com/adsr/mle
> >   License : Apache, Expat, BSD-1-clause
> >   Section : editors
> >
> > It builds those binary packages:
> >
> >   mle - flexible terminal-based text editor (C)
> >
> > To access further information about this package, please visit the 
> > following URL:
> >
> >   https://mentors.debian.net/package/mle
> >
> > Alternatively, one can download the package with dget using this command:
> >
> >   dget -x https://mentors.debian.net/debian/pool/main/m/mle/mle_1.2-1.dsc
> >
> > More information about hello can be obtained from 
> > https://github.com/adsr/mle.
> >
> > Changes since the last upload: -
> >
> > Regards,
> >
> > Adam
> >
>



Bug#910837: RFS: mle/1.2-1 [ITP] -- flexible terminal-based text editor (C)

2018-10-11 Thread Mo Zhou
Hi Adam,

Thanks for this package, the copyright file looks good to me.
However there are still a couple of problems:

1. Can we avoid shipping with embedded code copy[1] of lua, termbox and
   uthash? . lua 5.3 and uthash are already in the archive. termbox needs
   to be packaged.

2. I'd recommend you to create a packaging repo so that the others can
   track the changes about packaging with git. Since you are also the
   upstream author, it's fine if you set the source format as
   "3.0 (native)" and push the debian directory to the upstream git repo.
   A separate git repo following e.g. the gbp-buildpackage workflow is
   also acceptable. No packaging repo is fine too but not encouraged.

3. External tools such as fzf can be leveraged by mle to enhance
   features. That means those packages should be listed in the
   "Recommends" or "Suggests" fields.

Best.

[1] https://wiki.debian.org/EmbeddedCodeCopies

On Thu, Oct 11, 2018 at 08:04:33PM -0400, Adam Saponara wrote:
> Package: sponsorship-requests
> Severity: wishlist
> 
> Dear mentors,
> 
> I am looking for a sponsor for my package "mle":
> 
>   Package name: mle
>   Version : 1.2-1
>   Upstream Author : Adam Saponara 
>   URL : https://github.com/adsr/mle
>   License : Apache, Expat, BSD-1-clause
>   Section : editors
> 
> It builds those binary packages:
> 
>   mle - flexible terminal-based text editor (C)
> 
> To access further information about this package, please visit the following 
> URL:
> 
>   https://mentors.debian.net/package/mle
> 
> Alternatively, one can download the package with dget using this command:
> 
>   dget -x https://mentors.debian.net/debian/pool/main/m/mle/mle_1.2-1.dsc
> 
> More information about hello can be obtained from https://github.com/adsr/mle.
> 
> Changes since the last upload: -
> 
> Regards,
> 
> Adam
> 



Bug#910837: RFS: mle/1.2-1 [ITP] -- flexible terminal-based text editor (C)

2018-10-11 Thread Adam Saponara
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for a sponsor for my package "mle":

  Package name: mle
  Version : 1.2-1
  Upstream Author : Adam Saponara 
  URL : https://github.com/adsr/mle
  License : Apache, Expat, BSD-1-clause
  Section : editors

It builds those binary packages:

  mle - flexible terminal-based text editor (C)

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

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

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

  dget -x https://mentors.debian.net/debian/pool/main/m/mle/mle_1.2-1.dsc

More information about hello can be obtained from https://github.com/adsr/mle.

Changes since the last upload: -

Regards,

Adam