* [ITP] rpcbind 0.2.1 @ 2014-08-20 18:22 Pavel Fedin 2014-08-20 19:14 ` Corinna Vinschen ` (2 more replies) 0 siblings, 3 replies; 18+ messages in thread From: Pavel Fedin @ 2014-08-20 18:22 UTC (permalink / raw) To: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-20 18:22 [ITP] rpcbind 0.2.1 Pavel Fedin @ 2014-08-20 19:14 ` Corinna Vinschen 2014-08-21 14:42 ` Pavel Fedin 2014-08-22 10:18 ` Pavel Fedin 2014-08-21 2:04 ` Yaakov Selkowitz 2014-08-22 12:49 ` [ITP] rpcbind 0.2.1 Corinna Vinschen 2 siblings, 2 replies; 18+ messages in thread From: Corinna Vinschen @ 2014-08-20 19:14 UTC (permalink / raw) To: cygwin-apps [-- 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 --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-20 19:14 ` Corinna Vinschen @ 2014-08-21 14:42 ` Pavel Fedin 2014-08-21 15:05 ` Corinna Vinschen 2014-08-22 10:18 ` Pavel Fedin 1 sibling, 1 reply; 18+ messages in thread From: Pavel Fedin @ 2014-08-21 14:42 UTC (permalink / raw) To: Corinna Vinschen; +Cc: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 14:42 ` Pavel Fedin @ 2014-08-21 15:05 ` Corinna Vinschen 0 siblings, 0 replies; 18+ messages in thread From: Corinna Vinschen @ 2014-08-21 15:05 UTC (permalink / raw) To: cygwin-apps [-- 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 --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* RE: [ITP] rpcbind 0.2.1 2014-08-20 19:14 ` Corinna Vinschen 2014-08-21 14:42 ` Pavel Fedin @ 2014-08-22 10:18 ` Pavel Fedin 1 sibling, 0 replies; 18+ messages in thread From: Pavel Fedin @ 2014-08-22 10:18 UTC (permalink / raw) To: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-20 18:22 [ITP] rpcbind 0.2.1 Pavel Fedin 2014-08-20 19:14 ` Corinna Vinschen @ 2014-08-21 2:04 ` Yaakov Selkowitz 2014-08-21 14:54 ` Pavel Fedin 2014-08-21 14:57 ` Pavel Fedin 2014-08-22 12:49 ` [ITP] rpcbind 0.2.1 Corinna Vinschen 2 siblings, 2 replies; 18+ messages in thread From: Yaakov Selkowitz @ 2014-08-21 2:04 UTC (permalink / raw) To: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 2:04 ` Yaakov Selkowitz @ 2014-08-21 14:54 ` Pavel Fedin 2014-08-21 14:57 ` Pavel Fedin 1 sibling, 0 replies; 18+ messages in thread From: Pavel Fedin @ 2014-08-21 14:54 UTC (permalink / raw) To: Yaakov Selkowitz; +Cc: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 2:04 ` Yaakov Selkowitz 2014-08-21 14:54 ` Pavel Fedin @ 2014-08-21 14:57 ` Pavel Fedin 2014-08-21 15:09 ` Corinna Vinschen 1 sibling, 1 reply; 18+ messages in thread From: Pavel Fedin @ 2014-08-21 14:57 UTC (permalink / raw) To: Yaakov Selkowitz; +Cc: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 14:57 ` Pavel Fedin @ 2014-08-21 15:09 ` Corinna Vinschen 2014-08-21 16:01 ` Pavel Fedin 2014-08-21 16:53 ` Achim Gratz 0 siblings, 2 replies; 18+ messages in thread From: Corinna Vinschen @ 2014-08-21 15:09 UTC (permalink / raw) To: cygwin-apps [-- 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 --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 15:09 ` Corinna Vinschen @ 2014-08-21 16:01 ` Pavel Fedin 2014-08-21 16:34 ` Corinna Vinschen 2014-08-21 16:53 ` Achim Gratz 1 sibling, 1 reply; 18+ messages in thread From: Pavel Fedin @ 2014-08-21 16:01 UTC (permalink / raw) To: Corinna Vinschen; +Cc: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 16:01 ` Pavel Fedin @ 2014-08-21 16:34 ` Corinna Vinschen 0 siblings, 0 replies; 18+ messages in thread From: Corinna Vinschen @ 2014-08-21 16:34 UTC (permalink / raw) To: cygwin-apps [-- 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 --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 15:09 ` Corinna Vinschen 2014-08-21 16:01 ` Pavel Fedin @ 2014-08-21 16:53 ` Achim Gratz 2014-08-21 18:22 ` Corinna Vinschen 1 sibling, 1 reply; 18+ messages in thread From: Achim Gratz @ 2014-08-21 16:53 UTC (permalink / raw) To: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 16:53 ` Achim Gratz @ 2014-08-21 18:22 ` Corinna Vinschen 2014-08-21 18:47 ` Achim Gratz 0 siblings, 1 reply; 18+ messages in thread From: Corinna Vinschen @ 2014-08-21 18:22 UTC (permalink / raw) To: cygwin-apps [-- 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 --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 18:22 ` Corinna Vinschen @ 2014-08-21 18:47 ` Achim Gratz 2014-08-21 19:03 ` Corinna Vinschen 0 siblings, 1 reply; 18+ messages in thread From: Achim Gratz @ 2014-08-21 18:47 UTC (permalink / raw) To: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-21 18:47 ` Achim Gratz @ 2014-08-21 19:03 ` Corinna Vinschen 2014-08-21 19:13 ` additions to base-files (was: [ITP] rpcbind 0.2.1) Achim Gratz 0 siblings, 1 reply; 18+ messages in thread From: Corinna Vinschen @ 2014-08-21 19:03 UTC (permalink / raw) To: cygwin-apps [-- 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 --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: additions to base-files (was: [ITP] rpcbind 0.2.1) 2014-08-21 19:03 ` Corinna Vinschen @ 2014-08-21 19:13 ` Achim Gratz 0 siblings, 0 replies; 18+ messages in thread From: Achim Gratz @ 2014-08-21 19:13 UTC (permalink / raw) To: cygwin-apps 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-20 18:22 [ITP] rpcbind 0.2.1 Pavel Fedin 2014-08-20 19:14 ` Corinna Vinschen 2014-08-21 2:04 ` Yaakov Selkowitz @ 2014-08-22 12:49 ` Corinna Vinschen 2014-08-23 19:10 ` Pavel Fedin 2 siblings, 1 reply; 18+ messages in thread From: Corinna Vinschen @ 2014-08-22 12:49 UTC (permalink / raw) To: cygwin-apps [-- 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 --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [ITP] rpcbind 0.2.1 2014-08-22 12:49 ` [ITP] rpcbind 0.2.1 Corinna Vinschen @ 2014-08-23 19:10 ` Pavel Fedin 0 siblings, 0 replies; 18+ messages in thread From: Pavel Fedin @ 2014-08-23 19:10 UTC (permalink / raw) To: Corinna Vinschen 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2014-08-23 19:10 UTC | newest] Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-08-20 18:22 [ITP] rpcbind 0.2.1 Pavel Fedin 2014-08-20 19:14 ` Corinna Vinschen 2014-08-21 14:42 ` Pavel Fedin 2014-08-21 15:05 ` Corinna Vinschen 2014-08-22 10:18 ` Pavel Fedin 2014-08-21 2:04 ` Yaakov Selkowitz 2014-08-21 14:54 ` Pavel Fedin 2014-08-21 14:57 ` Pavel Fedin 2014-08-21 15:09 ` Corinna Vinschen 2014-08-21 16:01 ` Pavel Fedin 2014-08-21 16:34 ` Corinna Vinschen 2014-08-21 16:53 ` Achim Gratz 2014-08-21 18:22 ` Corinna Vinschen 2014-08-21 18:47 ` Achim Gratz 2014-08-21 19:03 ` Corinna Vinschen 2014-08-21 19:13 ` additions to base-files (was: [ITP] rpcbind 0.2.1) Achim Gratz 2014-08-22 12:49 ` [ITP] rpcbind 0.2.1 Corinna Vinschen 2014-08-23 19:10 ` Pavel Fedin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).