[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#822107: marked as done (RFS: eja/9.3.12-1 [ITP])

Your message dated Mon, 23 May 2016 10:46:48 +0000 (UTC)
with message-id <1353418483.1652467.1464000408369.JavaMail.yahoo@mail.yahoo.com>
and subject line Re: Bug#822107: RFS: eja/9.3.12-1 [ITP]
has caused the Debian Bug report #822107,
regarding RFS: eja/9.3.12-1 [ITP]
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
822107: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=822107
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "eja"

* Package name    : eja
  Version         : 9.3.12-1
  Upstream Author : Ubaldo Porcheddu
* URL             : https://github.com/ubaldus/eja
* License         : GPL
  Section         : web

It builds those binary packages:
eja - micro web server for static and dynamic Lua generated content.
To access further information about this package, please visit the following URL: 

  http://mentors.debian.net/package/eja
Alternatively, one can download the package with dget using this command:
dget -x http://mentors.debian.net/debian/pool/main/e/eja/eja_9.3.12-1.dsc
More information about hello can be obtained from https://github.com/ubaldus/eja. 


Regards,
   Ubaldo Porcheddu

--- End Message ---
--- Begin Message --- 
Hi,

>replaced with https://github.com/ubaldus/eja.git



nice
>updated to 3.9.8


ok
>done
>wrapped to 80 columns and I hope improved the description.
>removed


wonderful!

>ok, bug is #824919


wonderful

>I hope I added every licence and holder.


ok I think so, lets see ftpmasters' review



>I promise I will study cmake and try to improve :)

thanks!
>I hope I did it in the right way, I tried to compile with and without 
>already installed Lua and it seems to be working fine.


wonderful!
>I added the -g option but I could not find any way to run the dh_strip, 
>is this automatic?


yes!
>I hope I did it in the right way, it seems to be working fine on my 
>environment.

> 15) install something like this?
> override_dh_auto_install:
> dh_auto_install -- prefix=/usr

ok

>this file is needed by eja.c to "embed" the Lua files in lib/*.lua and 
>has to be created on the fly, I mean this is the only way I found out to 
>be working fine but of course I am open to any better suggestion.


ok, but not used in Debian builds, right?

>I am not sure if this is the right solution but I added them in 

>debian/rules.

your solution lacks of a build-dependency
(please try pbuilder/sbuild to see if the package builds in a clean environment)

I honestly prefer a simple
"export DEB_BUILD_MAINT_OPTIONS = hardening=+all"

> hope I have done it right.


missing the version string :)
(as shown on mentors page)
>replaced (was in json.lua).


ok
(please don't patch embedded libraries, or at least forward patches upstream too)
>should be better now.

ok
>should also be removed after eja.dirs deletion.

ok

>I tried the same and it complains about helpFull/helpful but indeed >helpFull is the name of the option meaning "full help".


ok

>honestly this library is really for /proc access but I found out I could 
>calculate the total amount of cores from /sys so I added it as first 
>option when counting them.


ok thanks



>> https://wiki.debian.org/UpstreamMetadata'

>I could not run these tests, I will try again later but if you already >have the output maybe it would help me to sort it out.


well, this is about adding a file with some content
https://codesearch.debian.net/results/path%3A%2Fmetadata%24%20Name/page_0

some new stuff to look at
(something has been already done by me, please look at the patch, integrate it, and look
at the differences between your version and mine



something I leave to you:
http://debomatic-amd64.debian.net/distribution#unstable/eja/9.5.20-1/lintian


cheers,

G.

Attachment: eja_9.5.20-1.debian.tar.xz
Description: application/xz


--- End Message ---
Reply to: