public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
* update_etc_shells gone from cygport?
@ 2016-08-29 12:40 Chris Sutcliffe
  2016-08-29 18:05 ` Yaakov Selkowitz
  0 siblings, 1 reply; 9+ messages in thread
From: Chris Sutcliffe @ 2016-08-29 12:40 UTC (permalink / raw)
  To: Cygwin-apps

Hi,

A new release of mksh was made so I fired up cygport to package it and
found out that "update_etc_shells" no longer seems to be part of
cygport (I'm running cygport 0.22.0).  Is there a new function that's
replaced "update_etc_shells"?

Thanks,

Chris

-- 
Chris Sutcliffe

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-29 12:40 update_etc_shells gone from cygport? Chris Sutcliffe
@ 2016-08-29 18:05 ` Yaakov Selkowitz
  2016-08-30 12:48   ` Corinna Vinschen
  0 siblings, 1 reply; 9+ messages in thread
From: Yaakov Selkowitz @ 2016-08-29 18:05 UTC (permalink / raw)
  To: cygwin-apps

On 2016-08-29 07:40, Chris Sutcliffe wrote:
> A new release of mksh was made so I fired up cygport to package it and
> found out that "update_etc_shells" no longer seems to be part of
> cygport (I'm running cygport 0.22.0).  Is there a new function that's
> replaced "update_etc_shells"?

It was just a proposed patch but got lost among other things.  I just 
pushed it into git master.

-- 
Yaakov

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-29 18:05 ` Yaakov Selkowitz
@ 2016-08-30 12:48   ` Corinna Vinschen
  2016-08-30 15:17     ` Yaakov Selkowitz
  0 siblings, 1 reply; 9+ messages in thread
From: Corinna Vinschen @ 2016-08-30 12:48 UTC (permalink / raw)
  To: cygwin-apps

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

On Aug 29 13:05, Yaakov Selkowitz wrote:
> On 2016-08-29 07:40, Chris Sutcliffe wrote:
> > A new release of mksh was made so I fired up cygport to package it and
> > found out that "update_etc_shells" no longer seems to be part of
> > cygport (I'm running cygport 0.22.0).  Is there a new function that's
> > replaced "update_etc_shells"?
> 
> It was just a proposed patch but got lost among other things.  I just pushed
> it into git master.

Can somebody please quickly explain what changes this requires in, say,
a potential tcsh cygport script?


Thanks,
Corinna

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

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-30 12:48   ` Corinna Vinschen
@ 2016-08-30 15:17     ` Yaakov Selkowitz
  2016-08-30 15:45       ` Corinna Vinschen
  0 siblings, 1 reply; 9+ messages in thread
From: Yaakov Selkowitz @ 2016-08-30 15:17 UTC (permalink / raw)
  To: cygwin-apps

On 2016-08-30 07:48, Corinna Vinschen wrote:
> On Aug 29 13:05, Yaakov Selkowitz wrote:
>> On 2016-08-29 07:40, Chris Sutcliffe wrote:
>>> A new release of mksh was made so I fired up cygport to package it and
>>> found out that "update_etc_shells" no longer seems to be part of
>>> cygport (I'm running cygport 0.22.0).  Is there a new function that's
>>> replaced "update_etc_shells"?
>>
>> It was just a proposed patch but got lost among other things.  I just pushed
>> it into git master.
>
> Can somebody please quickly explain what changes this requires in, say,
> a potential tcsh cygport script?

tcsh is actually the example in the documentation. :-)  Add the 
following to src_install():

update_etc_shells /bin/tcsh /bin/csh

But I think we need to coordinate this with an updated base-cygwin.

-- 
Yaakov

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-30 15:17     ` Yaakov Selkowitz
@ 2016-08-30 15:45       ` Corinna Vinschen
  2016-08-30 15:54         ` Yaakov Selkowitz
  2016-08-30 16:15         ` Achim Gratz
  0 siblings, 2 replies; 9+ messages in thread
From: Corinna Vinschen @ 2016-08-30 15:45 UTC (permalink / raw)
  To: cygwin-apps

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

On Aug 30 10:17, Yaakov Selkowitz wrote:
> On 2016-08-30 07:48, Corinna Vinschen wrote:
> > On Aug 29 13:05, Yaakov Selkowitz wrote:
> > > On 2016-08-29 07:40, Chris Sutcliffe wrote:
> > > > A new release of mksh was made so I fired up cygport to package it and
> > > > found out that "update_etc_shells" no longer seems to be part of
> > > > cygport (I'm running cygport 0.22.0).  Is there a new function that's
> > > > replaced "update_etc_shells"?
> > > 
> > > It was just a proposed patch but got lost among other things.  I just pushed
> > > it into git master.
> > 
> > Can somebody please quickly explain what changes this requires in, say,
> > a potential tcsh cygport script?
> 
> tcsh is actually the example in the documentation. :-)  Add the following to
> src_install():

"Documentation"?  Never heard about this.

> update_etc_shells /bin/tcsh /bin/csh
> 
> But I think we need to coordinate this with an updated base-cygwin.

base-cygwin?  Not base-files?


Corinna

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

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-30 15:45       ` Corinna Vinschen
@ 2016-08-30 15:54         ` Yaakov Selkowitz
  2016-08-30 16:15         ` Achim Gratz
  1 sibling, 0 replies; 9+ messages in thread
From: Yaakov Selkowitz @ 2016-08-30 15:54 UTC (permalink / raw)
  To: cygwin-apps

On 2016-08-30 10:45, Corinna Vinschen wrote:
> On Aug 30 10:17, Yaakov Selkowitz wrote:
>> But I think we need to coordinate this with an updated base-cygwin.
>
> base-cygwin?  Not base-files?

Sorry, you're right, base-files.

-- 
Yaakov

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-30 15:45       ` Corinna Vinschen
  2016-08-30 15:54         ` Yaakov Selkowitz
@ 2016-08-30 16:15         ` Achim Gratz
  2016-08-30 16:44           ` Yaakov Selkowitz
  1 sibling, 1 reply; 9+ messages in thread
From: Achim Gratz @ 2016-08-30 16:15 UTC (permalink / raw)
  To: cygwin-apps

Corinna Vinschen writes:
>> update_etc_shells /bin/tcsh /bin/csh
>> 
>> But I think we need to coordinate this with an updated base-cygwin.
>
> base-cygwin?  Not base-files?

The file /etc/shells is currently installed by base-files from
/etc/defaults/etc/shells unless it had been altered.  So if you invent a
mechanism to alter the contents of the file, /etc/shells will cease to
be handled in pre-remove and post-install of base-files automatically,
I'd think.  We could then remove it of course.


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

SD adaptations for Waldorf Q V3.00R3 and Q+ V3.54R2:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-30 16:15         ` Achim Gratz
@ 2016-08-30 16:44           ` Yaakov Selkowitz
  2016-08-31  8:29             ` Corinna Vinschen
  0 siblings, 1 reply; 9+ messages in thread
From: Yaakov Selkowitz @ 2016-08-30 16:44 UTC (permalink / raw)
  To: cygwin-apps

On 2016-08-30 11:15, Achim Gratz wrote:
> Corinna Vinschen writes:
>>> update_etc_shells /bin/tcsh /bin/csh
>>>
>>> But I think we need to coordinate this with an updated base-cygwin.
>>
>> base-cygwin?  Not base-files?
>
> The file /etc/shells is currently installed by base-files from
> /etc/defaults/etc/shells unless it had been altered.  So if you invent a
> mechanism to alter the contents of the file, /etc/shells will cease to
> be handled in pre-remove and post-install of base-files automatically,
> I'd think.  We could then remove it of course.

Yes, /etc/shells would end up staying once altered by update_etc_shells, 
but I don't see an issue with that.

-- 
Yaakov

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: update_etc_shells gone from cygport?
  2016-08-30 16:44           ` Yaakov Selkowitz
@ 2016-08-31  8:29             ` Corinna Vinschen
  0 siblings, 0 replies; 9+ messages in thread
From: Corinna Vinschen @ 2016-08-31  8:29 UTC (permalink / raw)
  To: cygwin-apps

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

On Aug 30 11:44, Yaakov Selkowitz wrote:
> On 2016-08-30 11:15, Achim Gratz wrote:
> > Corinna Vinschen writes:
> > > > update_etc_shells /bin/tcsh /bin/csh
> > > > 
> > > > But I think we need to coordinate this with an updated base-cygwin.
> > > 
> > > base-cygwin?  Not base-files?
> > 
> > The file /etc/shells is currently installed by base-files from
> > /etc/defaults/etc/shells unless it had been altered.  So if you invent a
> > mechanism to alter the contents of the file, /etc/shells will cease to
> > be handled in pre-remove and post-install of base-files automatically,
> > I'd think.  We could then remove it of course.
> 
> Yes, /etc/shells would end up staying once altered by update_etc_shells, but
> I don't see an issue with that.

Ok, guys, give the word when you do this.

Yaakov, if you want to do it during my absence I announced on
cygwin-developers, feel free to rebuild the tcsh package.  I pick up the
new cygport file when I'm back.


Thanks,
Corinna

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

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2016-08-31  8:29 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-29 12:40 update_etc_shells gone from cygport? Chris Sutcliffe
2016-08-29 18:05 ` Yaakov Selkowitz
2016-08-30 12:48   ` Corinna Vinschen
2016-08-30 15:17     ` Yaakov Selkowitz
2016-08-30 15:45       ` Corinna Vinschen
2016-08-30 15:54         ` Yaakov Selkowitz
2016-08-30 16:15         ` Achim Gratz
2016-08-30 16:44           ` Yaakov Selkowitz
2016-08-31  8:29             ` Corinna Vinschen

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).