Re: new: sysutils/htop

2016-11-16 Thread Juan Francisco Cantero Hurtado 
On Tue, Nov 15, 2016 at 06:05:32PM -0500, Ian Sutton wrote:
> On Tue, Nov 15, 2016 at 04:29:46PM -0500, Ian Sutton wrote:
> > I am happy to take over maintainership. I will investigate these issues
> > and incorporate your patches in a mail to follow. I'll try to get some
> > coredumps & fix extant issues too.
> 
> And here is the mail with tarball.
> 
> On Sat, Nov 05, 2016 at 10:24:23AM +0100, Landry Breuil wrote:
> > given that it runs for example on macppc.. ok to import with that line
> > zapped.
> 
> zapped
> 
> On Sat, Nov 05, 2016 at 12:03:32PM +0100, Juan Francisco Cantero Hurtado 
> wrote:
> > Random crashes and double frees. Run the program with MALLOC_OPTIONS=F.
> 
> I'll fix any that I find
>  
> > If you take the maintainership, I will import the port.
> 
> Happy to do this, although I do not have a CVS account

You don't need a CVS account to take the maintaineship of a package.
Just take care of updates and bug reports, that's all. Any committer can
commit your changes.

I've added a MAINTAINER line with your name and email address and
imported the port. Let me know if your prefer a different address for
the port.


> 
> > A few comments:
> > 
> > - Remove the extra space at the end of DESCR.
> > - Change "This is htop, an interactive" to "htop is an interactive" in
> >   DESCR.
> > - Remove the second phrase in DESCR. Users don't need to know what are
> >   the dependencies of the packages.
> 
> Fixed these three
> 
> > - Look the Makefile in my old port and copy the ALL_TARGET and
> >   INSTALL_STRIP lines to your Makefile, with the comment included.
> 
> Added
> 
> Ian



-- 
Juan Francisco Cantero Hurtado http://juanfra.info

Re: new: sysutils/htop

2016-11-15 Thread Ian Sutton 
On Tue, Nov 15, 2016 at 04:29:46PM -0500, Ian Sutton wrote:
> I am happy to take over maintainership. I will investigate these issues
> and incorporate your patches in a mail to follow. I'll try to get some
> coredumps & fix extant issues too.

And here is the mail with tarball.

On Sat, Nov 05, 2016 at 10:24:23AM +0100, Landry Breuil wrote:
> given that it runs for example on macppc.. ok to import with that line
> zapped.

zapped

On Sat, Nov 05, 2016 at 12:03:32PM +0100, Juan Francisco Cantero Hurtado wrote:
> Random crashes and double frees. Run the program with MALLOC_OPTIONS=F.

I'll fix any that I find
 
> If you take the maintainership, I will import the port.

Happy to do this, although I do not have a CVS account

> A few comments:
> 
> - Remove the extra space at the end of DESCR.
> - Change "This is htop, an interactive" to "htop is an interactive" in
>   DESCR.
> - Remove the second phrase in DESCR. Users don't need to know what are
>   the dependencies of the packages.

Fixed these three

> - Look the Makefile in my old port and copy the ALL_TARGET and
>   INSTALL_STRIP lines to your Makefile, with the comment included.

Added

Ian


htop-2.0.2.tgz
Description: application/tar-gz

Re: new: sysutils/htop

2016-11-15 Thread Ian Sutton 
On Sat, Nov 05, 2016 at 12:03:32PM +0100, Juan Francisco Cantero Hurtado wrote:
> On Sat, Nov 05, 2016 at 06:09:43AM -0400, Ian Sutton wrote:
> > On Sat, Nov 05, 2016 at 10:24:23AM +0100, Landry Breuil wrote:
> > > given that it runs for example on macppc.. ok to import with that line
> > > zapped.
> > 
> > zapped
> > 
> > On Sat, Nov 05, 2016 at 10:24:27AM +0100, Daniel Jakots wrote:
> > > You probably want to look at what juanfra@ already did: 
> > > https://github.com/jasperla/openbsd-wip/tree/master/sysutils/htop
> > 
> > hmm -- what's causing the commit hesitation?
> 
> Random crashes and double frees. Run the program with MALLOC_OPTIONS=F.
> 
> If you take the maintainership, I will import the port.
> 
> A few comments:
> 
> - Remove the extra space at the end of DESCR.
> - Change "This is htop, an interactive" to "htop is an interactive" in
>   DESCR.
> - Remove the second phrase in DESCR. Users don't need to know what are
>   the dependencies of the packages.
> - Look the Makefile in my old port and copy the ALL_TARGET and
>   INSTALL_STRIP lines to your Makefile, with the comment included.


Hi,

I am happy to take over maintainership. I will investigate these issues
and incorporate your patches in a mail to follow. I'll try to get some
coredumps & fix extant issues too.

Ian

Re: new: sysutils/htop

2016-11-05 Thread Juan Francisco Cantero Hurtado 
On Sat, Nov 05, 2016 at 06:09:43AM -0400, Ian Sutton wrote:
> On Sat, Nov 05, 2016 at 10:24:23AM +0100, Landry Breuil wrote:
> > given that it runs for example on macppc.. ok to import with that line
> > zapped.
> 
> zapped
> 
> On Sat, Nov 05, 2016 at 10:24:27AM +0100, Daniel Jakots wrote:
> > You probably want to look at what juanfra@ already did: 
> > https://github.com/jasperla/openbsd-wip/tree/master/sysutils/htop
> 
> hmm -- what's causing the commit hesitation?

Random crashes and double frees. Run the program with MALLOC_OPTIONS=F.

If you take the maintainership, I will import the port.

A few comments:

- Remove the extra space at the end of DESCR.
- Change "This is htop, an interactive" to "htop is an interactive" in
  DESCR.
- Remove the second phrase in DESCR. Users don't need to know what are
  the dependencies of the packages.
- Look the Makefile in my old port and copy the ALL_TARGET and
  INSTALL_STRIP lines to your Makefile, with the comment included.


-- 
Juan Francisco Cantero Hurtado http://juanfra.info

Re: new: sysutils/htop

2016-11-05 Thread Daniel Jakots 
On Sat, 5 Nov 2016 06:09:43 -0400, Ian Sutton  wrote:

> On Sat, Nov 05, 2016 at 10:24:27AM +0100, Daniel Jakots wrote:
> > You probably want to look at what juanfra@ already did: 
> > https://github.com/jasperla/openbsd-wip/tree/master/sysutils/htop  
> 
> hmm -- what's causing the commit hesitation?

See the discussion:
https://marc.info/?l=openbsd-ports=146283061630445=2

Re: new: sysutils/htop

2016-11-05 Thread Ian Sutton 
On Sat, Nov 05, 2016 at 10:24:23AM +0100, Landry Breuil wrote:
> given that it runs for example on macppc.. ok to import with that line
> zapped.

zapped

On Sat, Nov 05, 2016 at 10:24:27AM +0100, Daniel Jakots wrote:
> You probably want to look at what juanfra@ already did: 
> https://github.com/jasperla/openbsd-wip/tree/master/sysutils/htop

hmm -- what's causing the commit hesitation?

ian


htop-2.0.2.tgz
Description: application/tar-gz

Re: new: sysutils/htop

2016-11-05 Thread Landry Breuil 
On Sat, Nov 05, 2016 at 10:24:27AM +0100, Daniel Jakots wrote:
> On Sat, 5 Nov 2016 03:01:20 -0400, Ian Sutton  wrote:
> 
> > htop is an enhanced version of the venerable top(1).
> > 
> > could a porter please add this? i am more than happy to maintain.
> > 
> > ian
> 
> You probably want to look at what juanfra@ already did: 
> https://github.com/jasperla/openbsd-wip/tree/master/sysutils/htop

Yeah but his work is older - crashes still happen (i had one but
non-reproducible upon F2). That stuff can still be fixed in-tree..

Landry

Re: new: sysutils/htop

2016-11-05 Thread Daniel Jakots 
On Sat, 5 Nov 2016 03:01:20 -0400, Ian Sutton  wrote:

> htop is an enhanced version of the venerable top(1).
> 
> could a porter please add this? i am more than happy to maintain.
> 
> ian

You probably want to look at what juanfra@ already did: 
https://github.com/jasperla/openbsd-wip/tree/master/sysutils/htop

Cheers,
Daniel

Re: new: sysutils/htop

2016-11-05 Thread Landry Breuil 
On Sat, Nov 05, 2016 at 03:01:20AM -0400, Ian Sutton wrote:
> htop is an enhanced version of the venerable top(1).
> 
> could a porter please add this? i am more than happy to maintain.

works fine, and i see absolutely no reason for the ONLY_FOR_ARCHS line,
given that it runs for example on macppc.. ok to import with that line
zapped.

Landry

new: sysutils/htop

2016-11-05 Thread Ian Sutton 
htop is an enhanced version of the venerable top(1).

could a porter please add this? i am more than happy to maintain.

ian


htop-2.0.2.tgz
Description: application/tar-gz