Re: [ITP] rpcbind 0.2.1

2014-08-23 Thread 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



RE: [ITP] rpcbind 0.2.1

2014-08-22 Thread 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

2014-08-22 Thread Corinna Vinschen
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


pgpgchI4ypTei.pgp
Description: PGP signature


Re: [ITP] rpcbind 0.2.1

2014-08-21 Thread 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

2014-08-21 Thread 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

2014-08-21 Thread 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

2014-08-21 Thread Corinna Vinschen
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


pgp9iABlJgpsr.pgp
Description: PGP signature


Re: [ITP] rpcbind 0.2.1

2014-08-21 Thread Corinna Vinschen
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


pgpcD79fYEnLp.pgp
Description: PGP signature


Re: [ITP] rpcbind 0.2.1

2014-08-21 Thread 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

2014-08-21 Thread Corinna Vinschen
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


pgpmnlYKX1b9a.pgp
Description: PGP signature


Re: [ITP] rpcbind 0.2.1

2014-08-21 Thread 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

2014-08-21 Thread Corinna Vinschen
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


pgpyXAiTLoeLr.pgp
Description: PGP signature


Re: [ITP] rpcbind 0.2.1

2014-08-21 Thread 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

2014-08-21 Thread Corinna Vinschen
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


pgpeqZC1tkclF.pgp
Description: PGP signature


Re: [ITP] rpcbind 0.2.1

2014-08-20 Thread Corinna Vinschen
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


pgpWLkVu8OStP.pgp
Description: PGP signature


Re: [ITP] rpcbind 0.2.1

2014-08-20 Thread 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