Re: New port: Gaupol subtitle editor

2021-08-24 Thread Anindya Mukherjee
On Mon, Aug 16, 2021 at 12:15:23AM -0700, Anindya Mukherjee wrote:
> On Sun, Aug 15, 2021 at 12:21:45AM +0200, Stefan Hagen wrote:
> > Anindya Mukherjee wrote:
> > > On Sat, Aug 14, 2021 at 06:27:05PM +0200, Stefan Hagen wrote:
> > > > Anindya Mukherjee wrote:
> > > > > Ping
> > > > > 
> > > > > On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> > > > > > start COMMENT with lower-case please (unless it's a proper noun,
> > > > > > but that doesn't apply here).
> > > > > > 
> > > > > > we are a bit busy fixing hundreds of ports for a forthcoming
> > > > > > clang update at the moment so it may take a while to get to it :)
> > > > 
> > > > I reviewed this port. It works fine and it is a useful tool.
> > > > 
> > > > Remarks:
> > > > - NO_TEST=Yes should be set when there are no regression tests
> > > > - pkg/DESCR mentions README.aeidon.md, which is nowhere to be found
> > > > 
> > > > You can either install the file (to share/doc/), or link to the API
> > > > docs directly: https://otsaloma.io/gaupol/doc/api/aeidon.html
> > > > 
> > > > But actually, I think we can omit the paragraph about aeidon. If we want
> > > > it to be used by other software, we should package it independently from
> > > > the GTK UI. Right now, it's only used by gaupol and so I would not give
> > > > it special treatment.
> > > 
> > > Thanks a lot for reviewing the port. I have updated it following your
> > > suggestions above. Please have a look.
> > 
> > Looks good to me.
> > 
> > Thanks,
> > Stefan
> 
> Thanks! Looking forward to getting it committed.
> 
> Regards,
> Anindya

ping



Re: New port: Gaupol subtitle editor

2021-08-16 Thread Anindya Mukherjee
On Sun, Aug 15, 2021 at 12:21:45AM +0200, Stefan Hagen wrote:
> Anindya Mukherjee wrote:
> > On Sat, Aug 14, 2021 at 06:27:05PM +0200, Stefan Hagen wrote:
> > > Anindya Mukherjee wrote:
> > > > Ping
> > > > 
> > > > On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> > > > > start COMMENT with lower-case please (unless it's a proper noun,
> > > > > but that doesn't apply here).
> > > > > 
> > > > > we are a bit busy fixing hundreds of ports for a forthcoming
> > > > > clang update at the moment so it may take a while to get to it :)
> > > 
> > > I reviewed this port. It works fine and it is a useful tool.
> > > 
> > > Remarks:
> > > - NO_TEST=Yes should be set when there are no regression tests
> > > - pkg/DESCR mentions README.aeidon.md, which is nowhere to be found
> > > 
> > > You can either install the file (to share/doc/), or link to the API
> > > docs directly: https://otsaloma.io/gaupol/doc/api/aeidon.html
> > > 
> > > But actually, I think we can omit the paragraph about aeidon. If we want
> > > it to be used by other software, we should package it independently from
> > > the GTK UI. Right now, it's only used by gaupol and so I would not give
> > > it special treatment.
> > 
> > Thanks a lot for reviewing the port. I have updated it following your
> > suggestions above. Please have a look.
> 
> Looks good to me.
> 
> Thanks,
> Stefan

Thanks! Looking forward to getting it committed.

Regards,
Anindya



Re: New port: Gaupol subtitle editor

2021-08-14 Thread Stefan Hagen
Anindya Mukherjee wrote:
> On Sat, Aug 14, 2021 at 06:27:05PM +0200, Stefan Hagen wrote:
> > Anindya Mukherjee wrote:
> > > Ping
> > > 
> > > On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> > > > start COMMENT with lower-case please (unless it's a proper noun,
> > > > but that doesn't apply here).
> > > > 
> > > > we are a bit busy fixing hundreds of ports for a forthcoming
> > > > clang update at the moment so it may take a while to get to it :)
> > 
> > I reviewed this port. It works fine and it is a useful tool.
> > 
> > Remarks:
> > - NO_TEST=Yes should be set when there are no regression tests
> > - pkg/DESCR mentions README.aeidon.md, which is nowhere to be found
> > 
> > You can either install the file (to share/doc/), or link to the API
> > docs directly: https://otsaloma.io/gaupol/doc/api/aeidon.html
> > 
> > But actually, I think we can omit the paragraph about aeidon. If we want
> > it to be used by other software, we should package it independently from
> > the GTK UI. Right now, it's only used by gaupol and so I would not give
> > it special treatment.
> 
> Thanks a lot for reviewing the port. I have updated it following your
> suggestions above. Please have a look.

Looks good to me.

Thanks,
Stefan



Re: New port: Gaupol subtitle editor

2021-08-14 Thread Anindya Mukherjee
On Sat, Aug 14, 2021 at 06:27:05PM +0200, Stefan Hagen wrote:
> Anindya Mukherjee wrote:
> > Ping
> > 
> > On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> > > start COMMENT with lower-case please (unless it's a proper noun,
> > > but that doesn't apply here).
> > > 
> > > we are a bit busy fixing hundreds of ports for a forthcoming
> > > clang update at the moment so it may take a while to get to it :)
> 
> I reviewed this port. It works fine and it is a useful tool.
> 
> Remarks:
> - NO_TEST=Yes should be set when there are no regression tests
> - pkg/DESCR mentions README.aeidon.md, which is nowhere to be found
> 
> You can either install the file (to share/doc/), or link to the API
> docs directly: https://otsaloma.io/gaupol/doc/api/aeidon.html
> 
> But actually, I think we can omit the paragraph about aeidon. If we want
> it to be used by other software, we should package it independently from
> the GTK UI. Right now, it's only used by gaupol and so I would not give
> it special treatment.
> 
> Portcheck, make lib-depends-check are fine. 
> 
> Thanks for the work!
> 
> Stefan

Hi Stefan,

Thanks a lot for reviewing the port. I have updated it following your
suggestions above. Please have a look.

Regards,
Anindya


gaupol.tar.gz
Description: application/tar-gz


Re: New port: Gaupol subtitle editor

2021-08-14 Thread Stefan Hagen
Anindya Mukherjee wrote:
> Ping
> 
> On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> > start COMMENT with lower-case please (unless it's a proper noun,
> > but that doesn't apply here).
> > 
> > we are a bit busy fixing hundreds of ports for a forthcoming
> > clang update at the moment so it may take a while to get to it :)

I reviewed this port. It works fine and it is a useful tool.

Remarks:
- NO_TEST=Yes should be set when there are no regression tests
- pkg/DESCR mentions README.aeidon.md, which is nowhere to be found

You can either install the file (to share/doc/), or link to the API
docs directly: https://otsaloma.io/gaupol/doc/api/aeidon.html

But actually, I think we can omit the paragraph about aeidon. If we want
it to be used by other software, we should package it independently from
the GTK UI. Right now, it's only used by gaupol and so I would not give
it special treatment.

Portcheck, make lib-depends-check are fine. 

Thanks for the work!

Stefan



Re: New port: Gaupol subtitle editor

2021-08-12 Thread Anindya Mukherjee
Ping

On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> start COMMENT with lower-case please (unless it's a proper noun,
> but that doesn't apply here).
> 
> we are a bit busy fixing hundreds of ports for a forthcoming
> clang update at the moment so it may take a while to get to it :)
> 
> 
> On 2021/02/02 06:41, Anindya Mukherjee wrote:
> > Attached tarball.
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: February 1, 2021 10:31 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi, just to recap:
> > 
> > This package provides Gaupol, an editor for
> >  text-based subtitle files.
> > 
> > Maintainer: Anindya Mukherjee 
> > 
> > Homepage: https://otsaloma.io/gaupol/
> > 
> > Comments? OKs? I'm using it on my machine,
> > installed from pkg.
> > 
> > Thanks!
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 7:18 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Updated tarball attached. I have added myself as
> > maintainer.
> > 
> > Thanks,
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 12:04 AM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi,
> > 
> > Many thanks for the comments and fixes. Sure, I can take
> > maintainer. So, it looks like I only need to add maintainer
> > to your version? Build and package looks ok and runs.
> > Any other suggestions?
> > 
> > Thanks,
> > Anindya
> > 
> > From: Dimitri Karamazov 
> > Sent: January 28, 2021 9:14 PM
> > To: Anindya Mukherjee 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > > It supports multiple subtitle file formats and provides means of
> > > creating, editing and translating subtitles and timing subtitles to
> > > match video.
> > > 
> > > This is my first port, so there might be things I missed. I have tried
> > > to follow the porting guide closely. Please have a look and comment.
> > > 
> > > Thanks,
> > > Anindya
> > 
> > I've reattached with some changes
> > 
> > -py-* nomenclature is used for python modules not for standalone ports
> >  like this, so I've removed it.
> > -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
> >  in this case. 
> > -rearranged the dependencies in alpha order and added some missing
> >  runtime depends.
> > 
> > There are many tests available but most of those need xorg, access to which 
> > _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> > if MODPY_PYTEST is of any use if the tests don't run on their own.
> > 
> > Also I recommended you take maintainer since you're going to use it.
> > 
> > regards,
> >   Dimitri
> 
> 


gaupol.tar.gz
Description: application/tar-gz


Re: New port: Gaupol subtitle editor

2021-04-29 Thread Anindya Mukherjee
Ping

On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> start COMMENT with lower-case please (unless it's a proper noun,
> but that doesn't apply here).
> 
> we are a bit busy fixing hundreds of ports for a forthcoming
> clang update at the moment so it may take a while to get to it :)
> 
> 
> On 2021/02/02 06:41, Anindya Mukherjee wrote:
> > Attached tarball.
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: February 1, 2021 10:31 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi, just to recap:
> > 
> > This package provides Gaupol, an editor for
> >  text-based subtitle files.
> > 
> > Maintainer: Anindya Mukherjee 
> > 
> > Homepage: https://otsaloma.io/gaupol/
> > 
> > Comments? OKs? I'm using it on my machine,
> > installed from pkg.
> > 
> > Thanks!
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 7:18 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Updated tarball attached. I have added myself as
> > maintainer.
> > 
> > Thanks,
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 12:04 AM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi,
> > 
> > Many thanks for the comments and fixes. Sure, I can take
> > maintainer. So, it looks like I only need to add maintainer
> > to your version? Build and package looks ok and runs.
> > Any other suggestions?
> > 
> > Thanks,
> > Anindya
> > 
> > From: Dimitri Karamazov 
> > Sent: January 28, 2021 9:14 PM
> > To: Anindya Mukherjee 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > > It supports multiple subtitle file formats and provides means of
> > > creating, editing and translating subtitles and timing subtitles to
> > > match video.
> > > 
> > > This is my first port, so there might be things I missed. I have tried
> > > to follow the porting guide closely. Please have a look and comment.
> > > 
> > > Thanks,
> > > Anindya
> > 
> > I've reattached with some changes
> > 
> > -py-* nomenclature is used for python modules not for standalone ports
> >  like this, so I've removed it.
> > -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
> >  in this case. 
> > -rearranged the dependencies in alpha order and added some missing
> >  runtime depends.
> > 
> > There are many tests available but most of those need xorg, access to which 
> > _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> > if MODPY_PYTEST is of any use if the tests don't run on their own.
> > 
> > Also I recommended you take maintainer since you're going to use it.
> > 
> > regards,
> >   Dimitri
> 
> 


gaupol.tar.gz
Description: application/tar-gz


Re: New port: Gaupol subtitle editor

2021-03-29 Thread Anindya Mukherjee
Ping

On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> start COMMENT with lower-case please (unless it's a proper noun,
> but that doesn't apply here).
> 
> we are a bit busy fixing hundreds of ports for a forthcoming
> clang update at the moment so it may take a while to get to it :)
> 
> 
> On 2021/02/02 06:41, Anindya Mukherjee wrote:
> > Attached tarball.
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: February 1, 2021 10:31 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi, just to recap:
> > 
> > This package provides Gaupol, an editor for
> >  text-based subtitle files.
> > 
> > Maintainer: Anindya Mukherjee 
> > 
> > Homepage: https://otsaloma.io/gaupol/
> > 
> > Comments? OKs? I'm using it on my machine,
> > installed from pkg.
> > 
> > Thanks!
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 7:18 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Updated tarball attached. I have added myself as
> > maintainer.
> > 
> > Thanks,
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 12:04 AM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi,
> > 
> > Many thanks for the comments and fixes. Sure, I can take
> > maintainer. So, it looks like I only need to add maintainer
> > to your version? Build and package looks ok and runs.
> > Any other suggestions?
> > 
> > Thanks,
> > Anindya
> > 
> > From: Dimitri Karamazov 
> > Sent: January 28, 2021 9:14 PM
> > To: Anindya Mukherjee 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > > It supports multiple subtitle file formats and provides means of
> > > creating, editing and translating subtitles and timing subtitles to
> > > match video.
> > > 
> > > This is my first port, so there might be things I missed. I have tried
> > > to follow the porting guide closely. Please have a look and comment.
> > > 
> > > Thanks,
> > > Anindya
> > 
> > I've reattached with some changes
> > 
> > -py-* nomenclature is used for python modules not for standalone ports
> >  like this, so I've removed it.
> > -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
> >  in this case. 
> > -rearranged the dependencies in alpha order and added some missing
> >  runtime depends.
> > 
> > There are many tests available but most of those need xorg, access to which 
> > _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> > if MODPY_PYTEST is of any use if the tests don't run on their own.
> > 
> > Also I recommended you take maintainer since you're going to use it.
> > 
> > regards,
> >   Dimitri
> 
> 


gaupol.tar.gz
Description: application/tar-gz


Re: New port: Gaupol subtitle editor

2021-03-12 Thread Anindya Mukherjee
Ping

On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> start COMMENT with lower-case please (unless it's a proper noun,
> but that doesn't apply here).
> 
> we are a bit busy fixing hundreds of ports for a forthcoming
> clang update at the moment so it may take a while to get to it :)
> 
> 
> On 2021/02/02 06:41, Anindya Mukherjee wrote:
> > Attached tarball.
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: February 1, 2021 10:31 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi, just to recap:
> > 
> > This package provides Gaupol, an editor for
> >  text-based subtitle files.
> > 
> > Maintainer: Anindya Mukherjee 
> > 
> > Homepage: https://otsaloma.io/gaupol/
> > 
> > Comments? OKs? I'm using it on my machine,
> > installed from pkg.
> > 
> > Thanks!
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 7:18 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Updated tarball attached. I have added myself as
> > maintainer.
> > 
> > Thanks,
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 12:04 AM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi,
> > 
> > Many thanks for the comments and fixes. Sure, I can take
> > maintainer. So, it looks like I only need to add maintainer
> > to your version? Build and package looks ok and runs.
> > Any other suggestions?
> > 
> > Thanks,
> > Anindya
> > 
> > From: Dimitri Karamazov 
> > Sent: January 28, 2021 9:14 PM
> > To: Anindya Mukherjee 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > > It supports multiple subtitle file formats and provides means of
> > > creating, editing and translating subtitles and timing subtitles to
> > > match video.
> > > 
> > > This is my first port, so there might be things I missed. I have tried
> > > to follow the porting guide closely. Please have a look and comment.
> > > 
> > > Thanks,
> > > Anindya
> > 
> > I've reattached with some changes
> > 
> > -py-* nomenclature is used for python modules not for standalone ports
> >  like this, so I've removed it.
> > -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
> >  in this case. 
> > -rearranged the dependencies in alpha order and added some missing
> >  runtime depends.
> > 
> > There are many tests available but most of those need xorg, access to which 
> > _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> > if MODPY_PYTEST is of any use if the tests don't run on their own.
> > 
> > Also I recommended you take maintainer since you're going to use it.
> > 
> > regards,
> >   Dimitri
> 
> 


gaupol.tar.gz
Description: application/tar-gz


Re: New port: Gaupol subtitle editor

2021-02-12 Thread Anindya Mukherjee
Hi, reattaching the tarball for convenience, thanks!

Anindya

On Tue, Feb 02, 2021 at 11:50:40AM +, Stuart Henderson wrote:
> start COMMENT with lower-case please (unless it's a proper noun,
> but that doesn't apply here).
> 
> we are a bit busy fixing hundreds of ports for a forthcoming
> clang update at the moment so it may take a while to get to it :)
> 
> 
> On 2021/02/02 06:41, Anindya Mukherjee wrote:
> > Attached tarball.
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: February 1, 2021 10:31 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi, just to recap:
> > 
> > This package provides Gaupol, an editor for
> >  text-based subtitle files.
> > 
> > Maintainer: Anindya Mukherjee 
> > 
> > Homepage: https://otsaloma.io/gaupol/
> > 
> > Comments? OKs? I'm using it on my machine,
> > installed from pkg.
> > 
> > Thanks!
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 7:18 PM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Updated tarball attached. I have added myself as
> > maintainer.
> > 
> > Thanks,
> > Anindya
> > 
> > 
> > From: Anindya Mukherjee 
> > Sent: January 29, 2021 12:04 AM
> > To: Dimitri Karamazov 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > Hi,
> > 
> > Many thanks for the comments and fixes. Sure, I can take
> > maintainer. So, it looks like I only need to add maintainer
> > to your version? Build and package looks ok and runs.
> > Any other suggestions?
> > 
> > Thanks,
> > Anindya
> > 
> > From: Dimitri Karamazov 
> > Sent: January 28, 2021 9:14 PM
> > To: Anindya Mukherjee 
> > Cc: ports@openbsd.org 
> > Subject: Re: New port: Gaupol subtitle editor 
> >  
> > On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > > It supports multiple subtitle file formats and provides means of
> > > creating, editing and translating subtitles and timing subtitles to
> > > match video.
> > > 
> > > This is my first port, so there might be things I missed. I have tried
> > > to follow the porting guide closely. Please have a look and comment.
> > > 
> > > Thanks,
> > > Anindya
> > 
> > I've reattached with some changes
> > 
> > -py-* nomenclature is used for python modules not for standalone ports
> >  like this, so I've removed it.
> > -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
> >  in this case. 
> > -rearranged the dependencies in alpha order and added some missing
> >  runtime depends.
> > 
> > There are many tests available but most of those need xorg, access to which 
> > _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> > if MODPY_PYTEST is of any use if the tests don't run on their own.
> > 
> > Also I recommended you take maintainer since you're going to use it.
> > 
> > regards,
> >   Dimitri
> 
> 


gaupol.tar.gz
Description: application/tar-gz


Re: New port: Gaupol subtitle editor

2021-02-02 Thread Anindya Mukherjee
Hi,

Thanks! I've fixed the comment.

Regards,
Anindya

From: Stuart Henderson 
Sent: February 2, 2021 3:50 AM
To: Anindya Mukherjee 
Cc: Dimitri Karamazov ; ports@openbsd.org 

Subject: Re: New port: Gaupol subtitle editor 
 
start COMMENT with lower-case please (unless it's a proper noun,
but that doesn't apply here).

we are a bit busy fixing hundreds of ports for a forthcoming
clang update at the moment so it may take a while to get to it :)


On 2021/02/02 06:41, Anindya Mukherjee wrote:
> Attached tarball.
> 
> 
> From: Anindya Mukherjee 
> Sent: February 1, 2021 10:31 PM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> Hi, just to recap:
> 
> This package provides Gaupol, an editor for
>  text-based subtitle files.
> 
> Maintainer: Anindya Mukherjee 
> 
> Homepage: https://otsaloma.io/gaupol/
> 
> Comments? OKs? I'm using it on my machine,
> installed from pkg.
> 
> Thanks!
> Anindya
> 
> 
> From: Anindya Mukherjee 
> Sent: January 29, 2021 7:18 PM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> Updated tarball attached. I have added myself as
> maintainer.
> 
> Thanks,
> Anindya
> 
> 
> From: Anindya Mukherjee 
> Sent: January 29, 2021 12:04 AM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> Hi,
> 
> Many thanks for the comments and fixes. Sure, I can take
> maintainer. So, it looks like I only need to add maintainer
> to your version? Build and package looks ok and runs.
> Any other suggestions?
> 
> Thanks,
> Anindya
> 
> From: Dimitri Karamazov 
> Sent: January 28, 2021 9:14 PM
> To: Anindya Mukherjee 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > It supports multiple subtitle file formats and provides means of
> > creating, editing and translating subtitles and timing subtitles to
> > match video.
> > 
> > This is my first port, so there might be things I missed. I have tried
> > to follow the porting guide closely. Please have a look and comment.
> > 
> > Thanks,
> > Anindya
> 
> I've reattached with some changes
> 
> -py-* nomenclature is used for python modules not for standalone ports
>  like this, so I've removed it.
> -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
>  in this case. 
> -rearranged the dependencies in alpha order and added some missing
>  runtime depends.
> 
> There are many tests available but most of those need xorg, access to which 
> _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> if MODPY_PYTEST is of any use if the tests don't run on their own.
> 
> Also I recommended you take maintainer since you're going to use it.
> 
> regards,
>   Dimitri



gaupol.tar.gz
Description: gaupol.tar.gz


Re: New port: Gaupol subtitle editor

2021-02-02 Thread Dimitri Karamazov
On Tue, Feb 02, 2021 at 06:41:35AM +, Anindya Mukherjee wrote:
> Attached tarball.
> 
You've to wait for someone with commit rights to review this.
Eventually someone will take notice so no need to wait on this.
Just bump this thread with your tarball every one or two weeks.

There is a naming problem with your tarball. It is listed as
gaupol.tar.gz but it comes out as L6qkNDpQ.gz.
While in the parent(category) dir containing the port, do
`tar xcf gaupol.tar.gz gaupol`
then attach it. Don't rename it in your mail client if that is
what you're doing.

Build & run tested by me, reattached for convenience.

regards,
  Dimitri
> 
> From: Anindya Mukherjee 
> Sent: February 1, 2021 10:31 PM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
> ?
> Hi, just to recap:
> 
> This package provides Gaupol, an editor for
> ?text-based subtitle files.
> 
> Maintainer: Anindya Mukherjee 
> 
> Homepage: https://otsaloma.io/gaupol/
> 
> Comments? OKs? I'm using it on my machine,
> installed from pkg.
> 
> Thanks!
> Anindya
> 
> 
> From: Anindya Mukherjee 
> Sent: January 29, 2021 7:18 PM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
> ?
> Updated tarball attached. I have added myself as
> maintainer.
> 
> Thanks,
> Anindya
> 
> 
> From: Anindya Mukherjee 
> Sent: January 29, 2021 12:04 AM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
> ?
> Hi,
> 
> Many thanks for the comments and fixes. Sure, I can take
> maintainer. So, it looks like I only need to add maintainer
> to your version? Build and package looks ok and runs.
> Any other suggestions?
> 
> Thanks,
> Anindya
> 
> From: Dimitri Karamazov 
> Sent: January 28, 2021 9:14 PM
> To: Anindya Mukherjee 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
> ?
> On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > It supports multiple subtitle file formats and provides means of
> > creating, editing and translating subtitles and timing subtitles to
> > match video.
> > 
> > This is my first port, so there might be things I missed. I have tried
> > to follow the porting guide closely. Please have a look and comment.
> > 
> > Thanks,
> > Anindya
> 
> I've reattached with some changes
> 
> -py-* nomenclature is used for python modules not for standalone ports
> ?like this, so I've removed it.
> -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
> ?in this case. 
> -rearranged the dependencies in alpha order and added some missing
> ?runtime depends.
> 
> There are many tests available but most of those need xorg, access to which 
> _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> if MODPY_PYTEST is of any use if the tests don't run on their own.
> 
> Also I recommended you take maintainer since you're going to use it.
> 
> regards,
> ? Dimitri




gaupol.tar.gz
Description: Binary data


Re: New port: Gaupol subtitle editor

2021-02-02 Thread Stuart Henderson
start COMMENT with lower-case please (unless it's a proper noun,
but that doesn't apply here).

we are a bit busy fixing hundreds of ports for a forthcoming
clang update at the moment so it may take a while to get to it :)


On 2021/02/02 06:41, Anindya Mukherjee wrote:
> Attached tarball.
> 
> 
> From: Anindya Mukherjee 
> Sent: February 1, 2021 10:31 PM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> Hi, just to recap:
> 
> This package provides Gaupol, an editor for
>  text-based subtitle files.
> 
> Maintainer: Anindya Mukherjee 
> 
> Homepage: https://otsaloma.io/gaupol/
> 
> Comments? OKs? I'm using it on my machine,
> installed from pkg.
> 
> Thanks!
> Anindya
> 
> 
> From: Anindya Mukherjee 
> Sent: January 29, 2021 7:18 PM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> Updated tarball attached. I have added myself as
> maintainer.
> 
> Thanks,
> Anindya
> 
> 
> From: Anindya Mukherjee 
> Sent: January 29, 2021 12:04 AM
> To: Dimitri Karamazov 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> Hi,
> 
> Many thanks for the comments and fixes. Sure, I can take
> maintainer. So, it looks like I only need to add maintainer
> to your version? Build and package looks ok and runs.
> Any other suggestions?
> 
> Thanks,
> Anindya
> 
> From: Dimitri Karamazov 
> Sent: January 28, 2021 9:14 PM
> To: Anindya Mukherjee 
> Cc: ports@openbsd.org 
> Subject: Re: New port: Gaupol subtitle editor 
>  
> On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> > Hi, this is a port of gaupol, an editor for text-based subtitle files.
> > It supports multiple subtitle file formats and provides means of
> > creating, editing and translating subtitles and timing subtitles to
> > match video.
> > 
> > This is my first port, so there might be things I missed. I have tried
> > to follow the porting guide closely. Please have a look and comment.
> > 
> > Thanks,
> > Anindya
> 
> I've reattached with some changes
> 
> -py-* nomenclature is used for python modules not for standalone ports
>  like this, so I've removed it.
> -FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
>  in this case. 
> -rearranged the dependencies in alpha order and added some missing
>  runtime depends.
> 
> There are many tests available but most of those need xorg, access to which 
> _pbuild doesn't have? One can run them as a normal user though. I'm not sure
> if MODPY_PYTEST is of any use if the tests don't run on their own.
> 
> Also I recommended you take maintainer since you're going to use it.
> 
> regards,
>   Dimitri




Re: New port: Gaupol subtitle editor

2021-02-01 Thread Anindya Mukherjee
Attached tarball.


From: Anindya Mukherjee 
Sent: February 1, 2021 10:31 PM
To: Dimitri Karamazov 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
Hi, just to recap:

This package provides Gaupol, an editor for
 text-based subtitle files.

Maintainer: Anindya Mukherjee 

Homepage: https://otsaloma.io/gaupol/

Comments? OKs? I'm using it on my machine,
installed from pkg.

Thanks!
Anindya


From: Anindya Mukherjee 
Sent: January 29, 2021 7:18 PM
To: Dimitri Karamazov 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
Updated tarball attached. I have added myself as
maintainer.

Thanks,
Anindya


From: Anindya Mukherjee 
Sent: January 29, 2021 12:04 AM
To: Dimitri Karamazov 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
Hi,

Many thanks for the comments and fixes. Sure, I can take
maintainer. So, it looks like I only need to add maintainer
to your version? Build and package looks ok and runs.
Any other suggestions?

Thanks,
Anindya

From: Dimitri Karamazov 
Sent: January 28, 2021 9:14 PM
To: Anindya Mukherjee 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> Hi, this is a port of gaupol, an editor for text-based subtitle files.
> It supports multiple subtitle file formats and provides means of
> creating, editing and translating subtitles and timing subtitles to
> match video.
> 
> This is my first port, so there might be things I missed. I have tried
> to follow the porting guide closely. Please have a look and comment.
> 
> Thanks,
> Anindya

I've reattached with some changes

-py-* nomenclature is used for python modules not for standalone ports
 like this, so I've removed it.
-FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
 in this case. 
-rearranged the dependencies in alpha order and added some missing
 runtime depends.

There are many tests available but most of those need xorg, access to which 
_pbuild doesn't have? One can run them as a normal user though. I'm not sure
if MODPY_PYTEST is of any use if the tests don't run on their own.

Also I recommended you take maintainer since you're going to use it.

regards,
  Dimitri

gaupol.tar.gz
Description: gaupol.tar.gz


Re: New port: Gaupol subtitle editor

2021-02-01 Thread Anindya Mukherjee
Hi, just to recap:

This package provides Gaupol, an editor for
 text-based subtitle files.

Maintainer: Anindya Mukherjee 

Homepage: https://otsaloma.io/gaupol/

Comments? OKs? I'm using it on my machine,
installed from pkg.

Thanks!
Anindya


From: Anindya Mukherjee 
Sent: January 29, 2021 7:18 PM
To: Dimitri Karamazov 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
Updated tarball attached. I have added myself as
maintainer.

Thanks,
Anindya


From: Anindya Mukherjee 
Sent: January 29, 2021 12:04 AM
To: Dimitri Karamazov 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
Hi,

Many thanks for the comments and fixes. Sure, I can take
maintainer. So, it looks like I only need to add maintainer
to your version? Build and package looks ok and runs.
Any other suggestions?

Thanks,
Anindya

From: Dimitri Karamazov 
Sent: January 28, 2021 9:14 PM
To: Anindya Mukherjee 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> Hi, this is a port of gaupol, an editor for text-based subtitle files.
> It supports multiple subtitle file formats and provides means of
> creating, editing and translating subtitles and timing subtitles to
> match video.
> 
> This is my first port, so there might be things I missed. I have tried
> to follow the porting guide closely. Please have a look and comment.
> 
> Thanks,
> Anindya

I've reattached with some changes

-py-* nomenclature is used for python modules not for standalone ports
 like this, so I've removed it.
-FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
 in this case. 
-rearranged the dependencies in alpha order and added some missing
 runtime depends.

There are many tests available but most of those need xorg, access to which 
_pbuild doesn't have? One can run them as a normal user though. I'm not sure
if MODPY_PYTEST is of any use if the tests don't run on their own.

Also I recommended you take maintainer since you're going to use it.

regards,
  Dimitri


Re: New port: Gaupol subtitle editor

2021-01-29 Thread Anindya Mukherjee
Updated tarball attached. I have added myself as
maintainer.

Thanks,
Anindya


From: Anindya Mukherjee 
Sent: January 29, 2021 12:04 AM
To: Dimitri Karamazov 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
Hi,

Many thanks for the comments and fixes. Sure, I can take
maintainer. So, it looks like I only need to add maintainer
to your version? Build and package looks ok and runs.
Any other suggestions?

Thanks,
Anindya

From: Dimitri Karamazov 
Sent: January 28, 2021 9:14 PM
To: Anindya Mukherjee 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> Hi, this is a port of gaupol, an editor for text-based subtitle files.
> It supports multiple subtitle file formats and provides means of
> creating, editing and translating subtitles and timing subtitles to
> match video.
> 
> This is my first port, so there might be things I missed. I have tried
> to follow the porting guide closely. Please have a look and comment.
> 
> Thanks,
> Anindya

I've reattached with some changes

-py-* nomenclature is used for python modules not for standalone ports
 like this, so I've removed it.
-FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
 in this case. 
-rearranged the dependencies in alpha order and added some missing
 runtime depends.

There are many tests available but most of those need xorg, access to which 
_pbuild doesn't have? One can run them as a normal user though. I'm not sure
if MODPY_PYTEST is of any use if the tests don't run on their own.

Also I recommended you take maintainer since you're going to use it.

regards,
  Dimitri

gaupol.tar.gz
Description: gaupol.tar.gz


Re: New port: Gaupol subtitle editor

2021-01-29 Thread Dimitri Karamazov
On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> Hi, this is a port of gaupol, an editor for text-based subtitle files.
> It supports multiple subtitle file formats and provides means of
> creating, editing and translating subtitles and timing subtitles to
> match video.
> 
> This is my first port, so there might be things I missed. I have tried
> to follow the porting guide closely. Please have a look and comment.
> 
> Thanks,
> Anindya

I've reattached with some changes

-py-* nomenclature is used for python modules not for standalone ports
 like this, so I've removed it.
-FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
 in this case. 
-rearranged the dependencies in alpha order and added some missing
 runtime depends.

There are many tests available but most of those need xorg, access to which 
_pbuild doesn't have? One can run them as a normal user though. I'm not sure
if MODPY_PYTEST is of any use if the tests don't run on their own.

Also I recommended you take maintainer since you're going to use it.

regards,
  Dimitri


gaupol.tar.gz
Description: Binary data


Re: New port: Gaupol subtitle editor

2021-01-29 Thread Anindya Mukherjee
Hi,

Many thanks for the comments and fixes. Sure, I can take
maintainer. So, it looks like I only need to add maintainer
to your version? Build and package looks ok and runs.
Any other suggestions?

Thanks,
Anindya

From: Dimitri Karamazov 
Sent: January 28, 2021 9:14 PM
To: Anindya Mukherjee 
Cc: ports@openbsd.org 
Subject: Re: New port: Gaupol subtitle editor 
 
On Fri, Jan 29, 2021 at 04:28:50AM +, Dimitri Karamazov wrote:
> Hi, this is a port of gaupol, an editor for text-based subtitle files.
> It supports multiple subtitle file formats and provides means of
> creating, editing and translating subtitles and timing subtitles to
> match video.
> 
> This is my first port, so there might be things I missed. I have tried
> to follow the porting guide closely. Please have a look and comment.
> 
> Thanks,
> Anindya

I've reattached with some changes

-py-* nomenclature is used for python modules not for standalone ports
 like this, so I've removed it.
-FLAVOR{s,} is also used for python modules only, MODPY_VERSION is suffice
 in this case. 
-rearranged the dependencies in alpha order and added some missing
 runtime depends.

There are many tests available but most of those need xorg, access to which 
_pbuild doesn't have? One can run them as a normal user though. I'm not sure
if MODPY_PYTEST is of any use if the tests don't run on their own.

Also I recommended you take maintainer since you're going to use it.

regards,
  Dimitri


New port: Gaupol subtitle editor

2021-01-28 Thread Anindya Mukherjee
Hi, this is a port of gaupol, an editor for text-based subtitle files.
It supports multiple subtitle file formats and provides means of
creating, editing and translating subtitles and timing subtitles to
match video.

This is my first port, so there might be things I missed. I have tried
to follow the porting guide closely. Please have a look and comment.

Thanks,
Anindya


multimedia-py-gaupol.tar.gz
Description: multimedia-py-gaupol.tar.gz