[ITP] rpcbind 0.2.1

[ITP] rpcbind 0.2.1

[Pavel Fedin]

 Hello!
 I  have  completed rpcbind package and would like to offer it for the
upload.
https://www.dropbox.com/sh/ah2v9q1we8wl1oe/AACHiGJcCp6-oBrL2KEn7E7Wa?dl=0
 The same as onc-rpc-devel, this is x86-64-only for now.

-- 
Kind regards,
 Pavel

Re: [ITP] rpcbind 0.2.1

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 848 bytes --]

Hi Pavel,

On Aug 20 22:22, Pavel Fedin wrote:
>  Hello!
>  I  have  completed rpcbind package and would like to offer it for the
> upload.
> https://www.dropbox.com/sh/ah2v9q1we8wl1oe/AACHiGJcCp6-oBrL2KEn7E7Wa?dl=0
>  The same as onc-rpc-devel, this is x86-64-only for now.

Packaging looks good, basically, but the executables are in /usr/bin,
while on Linux they are in /usr/sbin.  Would you mind to move them?

Btw., can you give a quick overview what's going to happen with the
still missing executables from the old sunrpc package. /usr/sbin/portmap
and /usr/bin/rstat?  You probably explained that already at one point
but I seem to have missed that :}


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

Re: [ITP] rpcbind 0.2.1

[Yaakov Selkowitz]

On 2014-08-20 13:22, Pavel Fedin wrote:
>   Hello!
>   I  have  completed rpcbind package and would like to offer it for the
> upload.
> https://www.dropbox.com/sh/ah2v9q1we8wl1oe/AACHiGJcCp6-oBrL2KEn7E7Wa?dl=0
>   The same as onc-rpc-devel, this is x86-64-only for now.

Comments follow.

> OBSOLETES="sunrpc"

I'm wondering if onc-rpc-devel (which I'm starting to think should just 
be called rpcgen) shouldn't obsolete sunrpc too instead of this.  It's a 
bit of a tough call because there is some overlap here.

> SRC_URI="http://sourceforge.net/projects/rpcbind/files/rpcbind/0.2.1/rpcbind-0.2.1.tar.bz2"

This should be:

SRC_URI="mirror://sourceforge/rpcbind/rpcbind-${VERSION}.tar.bz2"

> src_install () {
> 	cd ${B}
> 	cyginstall
> }

These are in /usr/sbin on Fedora, so in this particular case:

cyginstall bindir=/usr/sbin

> +#ifndef __CYGWIN__
>  	if (geteuid()) { /* This command allowed only to root */

Perhaps this is a candidate for using Daniel Boland's code?


> -			syslog(LOG_ERR, "cannot get uid of '%s': %m", id);
> +  			syslog(LOG_ERR, "cannot get uid of '%s': %m", id);

This is just a whitespace difference, I suggest you leave it out of your 
patch to ease future porting.


Yaakov

Re: [ITP] rpcbind 0.2.1

[Pavel Fedin]

Hello, Corinna.

Wednesday, August 20, 2014, 23:14:35 you wrote:

> Packaging looks good, basically, but the executables are in /usr/bin,
> while on Linux they are in /usr/sbin.  Would you mind to move them?

 No, i don't mind. I will.

> Btw., can you give a quick overview what's going to happen with the
> still missing executables from the old sunrpc package. /usr/sbin/portmap
> and /usr/bin/rstat?

 portmap is now rpcbind and rstat is now rpcinfo.
 rpcbind  is a new portmapper, designed with capabilities of TI-RPC in
mind. Old portmap doesn't live quite well with it. So the whole sunrpc
is completely obsolete.

-- 
Kind regards,
 Pavel

Re: [ITP] rpcbind 0.2.1

[Pavel Fedin]

Hello, Yaakov.

Thursday, August 21, 2014, 6:04:38 you wrote:

> I'm wondering if onc-rpc-devel (which I'm starting to think should just
> be called rpcgen) shouldn't obsolete sunrpc too instead of this.

 I  went  for  a  new name because rpcgen was just rpcgen, and the new
package is rpcgen + includes.
 Actually  initially  i  didn't  want to touch rpcgen at all, and just
make  onc-rpc-headers, however after that i thought that both packages
are  quite small, so it makes sense to merge them. After all, .x files
are  useless  without  rpcgen  and  vice  versa. And both are used for
development only.

>   It's a bit of a tough call because there is some overlap here.

 Original  sunrpc  package  consists  of  these  includes and portmap.
onc-rpc-devel now has includes, and rpcbind is a new portmap. So, as a
result, none of two new packages alone completely obsoletes sunrpc, but both
of them do. How to specify this ?

 As to the rest of notes, i'll fix that.

-- 
Kind regards,
 Pavel

Re: [ITP] rpcbind 0.2.1

[Pavel Fedin]

Hello, Yaakov.

Thursday, August 21, 2014, 6:04:38 you wrote:

> These are in /usr/sbin on Fedora, so in this particular case:

> cyginstall bindir=/usr/sbin

 By  the  way,  i  just have noticed that Cygwin actually doesn't have
/usr/sbin.  At  least  we  don't  have it in $PATH by default. For the
service  it's  okay, anyway we run it using cygrunsrv, but for rpcinfo
this would be inconvenient.

-- 
Kind regards,
 Pavel

Re: [ITP] rpcbind 0.2.1

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 908 bytes --]

On Aug 21 18:42, Pavel Fedin wrote:
> Hello, Corinna.
> 
> Wednesday, August 20, 2014, 23:14:35 you wrote:
> 
> > Packaging looks good, basically, but the executables are in /usr/bin,
> > while on Linux they are in /usr/sbin.  Would you mind to move them?
> 
>  No, i don't mind. I will.
> 
> > Btw., can you give a quick overview what's going to happen with the
> > still missing executables from the old sunrpc package. /usr/sbin/portmap
> > and /usr/bin/rstat?
> 
>  portmap is now rpcbind and rstat is now rpcinfo.
>  rpcbind  is a new portmapper, designed with capabilities of TI-RPC in
> mind. Old portmap doesn't live quite well with it. So the whole sunrpc
> is completely obsolete.

Uh, super.  Thanks for the info.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

Re: [ITP] rpcbind 0.2.1

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 881 bytes --]

On Aug 21 18:58, Pavel Fedin wrote:
> Hello, Yaakov.
> 
> Thursday, August 21, 2014, 6:04:38 you wrote:
> 
> > These are in /usr/sbin on Fedora, so in this particular case:
> 
> > cyginstall bindir=/usr/sbin
> 
>  By  the  way,  i  just have noticed that Cygwin actually doesn't have
> /usr/sbin.  At  least  we  don't  have it in $PATH by default. For the
> service  it's  okay, anyway we run it using cygrunsrv, but for rpcinfo
> this would be inconvenient.

That's a generic installation problem, not a problem of your package.

We can (and maybe should) consider to add /usr/sbin to $PATH by default
in /etc/profile, but that's a discussion concerning the base-files
package in the first place.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

Re: [ITP] rpcbind 0.2.1

[Pavel Fedin]

Hello, Corinna.

Thursday, August 21, 2014, 19:08:58 you wrote:

> That's a generic installation problem, not a problem of your package.

 Yes,  may  be,  however  this doesn't give any answer to: how will we
solve this particular problem right now ? There are variants:
1. I leave rpcinfo in /usr/bin
2. You (or maintainer ?) update base-files.
 So ?

-- 
Kind regards,
 Pavel

Re: [ITP] rpcbind 0.2.1

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 699 bytes --]

On Aug 21 19:58, Pavel Fedin wrote:
> Hello, Corinna.
> 
> Thursday, August 21, 2014, 19:08:58 you wrote:
> 
> > That's a generic installation problem, not a problem of your package.
> 
>  Yes,  may  be,  however  this doesn't give any answer to: how will we
> solve this particular problem right now ? There are variants:
> 1. I leave rpcinfo in /usr/bin
> 2. You (or maintainer ?) update base-files.
>  So ?

3. You put it into /usr/sbin independently of the state of base-files.

I don't see a blocking problem here.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

Re: [ITP] rpcbind 0.2.1

[Achim Gratz]

Corinna Vinschen writes:
> We can (and maybe should) consider to add /usr/sbin to $PATH by default
> in /etc/profile, but that's a discussion concerning the base-files
> package in the first place.

Only root should have it in PATH otherwise you could just install
everything in /usr/bin and forget about it – which gets us back to the
problem of who gets to call himself root in Cygwin.  BTW, /sbin and
/usr/sbin are currently separate – sysvinit and procps are the only
packages that install files into /sbin, though.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Waldorf MIDI Implementation & additional documentation:
http://Synth.Stromeko.net/Downloads.html#WaldorfDocs

Re: [ITP] rpcbind 0.2.1

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 1122 bytes --]

On Aug 21 18:53, Achim Gratz wrote:
> Corinna Vinschen writes:
> > We can (and maybe should) consider to add /usr/sbin to $PATH by default
> > in /etc/profile, but that's a discussion concerning the base-files
> > package in the first place.
> 
> Only root should have it in PATH otherwise you could just install
> everything in /usr/bin and forget about it – which gets us back to the
> problem of who gets to call himself root in Cygwin.  BTW, /sbin and
> /usr/sbin are currently separate – sysvinit and procps are the only
> packages that install files into /sbin, though.

Procps only for sysctl, and sysctl is not working on Cygwin anyway.
I'll create a new procps package which omits sysctl entirely.

As for sysvinit (and potential others), I'm not exactly keen to add
another hardcoded mount /usr/sbin -> /sbin or some such.  Perhaps
we should simply disregard /sbin and see to it that no new package
installs anything into it.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

Re: [ITP] rpcbind 0.2.1

[Achim Gratz]

Corinna Vinschen writes:
> Procps only for sysctl, and sysctl is not working on Cygwin anyway.
> I'll create a new procps package which omits sysctl entirely.

Great, thank you.

> As for sysvinit (and potential others), I'm not exactly keen to add
> another hardcoded mount /usr/sbin -> /sbin or some such.  Perhaps
> we should simply disregard /sbin and see to it that no new package
> installs anything into it.

OK, it is currently an outlier anyway.  What does one use sysvinit for
with cygwin I might ask?

Now back to the other question: do we want or need to provide some
standard / sanctioned notion of user root or group wheel?  I still have
those feature requests against base-files to change the shell prompt for
users in group 544 to "#".  It'd be easy enough to keep the result in a
variable for other uses.  on the other hand I didn't want to do it in
/etc/profile so far, but if it becomes standard fare that might be
necessary.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Wavetables for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldUserWavetables

Re: [ITP] rpcbind 0.2.1

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 1230 bytes --]

On Aug 21 20:47, Achim Gratz wrote:
> Corinna Vinschen writes:
> > Procps only for sysctl, and sysctl is not working on Cygwin anyway.
> > I'll create a new procps package which omits sysctl entirely.
> 
> Great, thank you.
> 
> > As for sysvinit (and potential others), I'm not exactly keen to add
> > another hardcoded mount /usr/sbin -> /sbin or some such.  Perhaps
> > we should simply disregard /sbin and see to it that no new package
> > installs anything into it.
> 
> OK, it is currently an outlier anyway.  What does one use sysvinit for
> with cygwin I might ask?

I have no idea.

> Now back to the other question: do we want or need to provide some
> standard / sanctioned notion of user root or group wheel?

We won't invent anything like that.  It will only make sense with
passwd and group files and those are going away in the long run.

> I still have
> those feature requests against base-files to change the shell prompt for
> users in group 544 to "#".

That's fine, but everything beyond that is strictly local setup.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

Re: additions to base-files (was: [ITP] rpcbind 0.2.1)

[Achim Gratz]

Corinna Vinschen writes:
>> I still have
>> those feature requests against base-files to change the shell prompt for
>> users in group 544 to "#".
>
> That's fine, but everything beyond that is strictly local setup.

I'll implement it in profile.d then and everybody who needs it will have
to pick it up from there.  I can include the addition of /usr/sbin
either together with setting the prompt or as another optional drop-in.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

SD adaptation for Waldorf rackAttack V1.04R1:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada

RE: [ITP] rpcbind 0.2.1

[Pavel Fedin]

 Hello!

 I have updated the package. UID check is still #ifdef'ed out because Yaakov's library is not released yet. Additionally, i believe that some additional code is perhaps a good solution for large complex projects, but this one is too simple.

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia

Re: [ITP] rpcbind 0.2.1

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 691 bytes --]

On Aug 20 22:22, Pavel Fedin wrote:
>  Hello!
>  I  have  completed rpcbind package and would like to offer it for the
> upload.
> https://www.dropbox.com/sh/ah2v9q1we8wl1oe/AACHiGJcCp6-oBrL2KEn7E7Wa?dl=0
>  The same as onc-rpc-devel, this is x86-64-only for now.

Looks good.  Btw., I think ou can drop this piece of your cygport file:

  src_install () {
	  cd ${B}
	  cyginstall
  }

This is the default behaviour when running

  cygport rpcbind.cygport install

But it also doesn't hurt, so GTG.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

Re: [ITP] rpcbind 0.2.1

[Pavel Fedin]

Hello, Corinna.

Friday, August 22, 2014, 16:49:31 you wrote:

>   src_install () {
>           cd ${B}
>           cyginstall
>   }

> This is the default behaviour when running

>   cygport rpcbind.cygport install

> But it also doesn't hurt, so GTG.

 Fixed and uploaded.

-- 
Kind regards,
 Pavel