public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
* Patch for shutdown
@ 2013-05-20 11:41 Frank Fesevur
  2013-05-20 13:21 ` Jeffrey Altman
  2013-05-21  9:42 ` Corinna Vinschen
  0 siblings, 2 replies; 15+ messages in thread
From: Frank Fesevur @ 2013-05-20 11:41 UTC (permalink / raw)
  To: cygwin

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

Hi Corinna,

Since nobody made real objections about changing the short flag for
hibernate from -h to -b to make room for -h for halt [1], I created a
patch to change the shutdown program. The patch was created back in
March, but with all your work being done on cygwin64, your holiday and
myself being very busy with life it was held back.

There were no change to the shutdown CVS since then so here is that
patch, including a changelog.

Note that I didn't change the programs version number in this patch. I
don't know if changing an existing option would require/justify to
change the version from 1.9 to 2.0 instead of simply to 1.10?

Regards,
Frank

[1] http://cygwin.com/ml/cygwin/2013-03/msg00354.html

[-- Attachment #2: CHANGELOG --]
[-- Type: application/octet-stream, Size: 427 bytes --]

2013-03-23  Frank Fesevur  <ffes@users.sourceforge.net>

	* Changed the short option for hibernate from -h to -b.
	* Added -h, --halt as additional option for shutdown and -c, --cancel as additional option for abort.
	* Updated shutdown.8 man pages to reflect these changes.
	* Fixed typo: changed text InitateSystemShutdown to InitiateSystemShutdownEx in the two usage() functions in shutdown.c and in shutdown.8 and reboot.8

[-- Attachment #3: shutdown.patch --]
[-- Type: application/octet-stream, Size: 5549 bytes --]

? CHANGELOG
? shutdown.patch
Index: reboot.8
===================================================================
RCS file: /cvs/cygwin-apps/shutdown/reboot.8,v
retrieving revision 1.1
diff -u -p -r1.1 reboot.8
--- reboot.8	21 Mar 2013 11:58:00 -0000	1.1
+++ reboot.8	25 Mar 2013 11:26:12 -0000
@@ -19,7 +19,7 @@ reboot, halt, poweroff, hibernate, suspe
 .B suspend
 [\fIoptions\fR]
 .SH DESCRIPTION
-These programs allow to
+These programs allow the user to
 .BR reboot ", " halt ", " poweroff ", " hibernate " or " suspend
 the system.
 .PP
@@ -30,7 +30,7 @@ They are aliases for \fBshutdown --reboo
 Forces the execution. This can cause applications to lose data.
 .TP
 \fB\-x\fR, \fB\-\-exitex\fR
-Use ExitWindowsEx rather than InitateSystemShutdown.
+Use ExitWindowsEx rather than InitiateSystemShutdownEx.
 .TP
 \fB\-\-help\fR
 Display a help text and exit.
Index: shutdown.8
===================================================================
RCS file: /cvs/cygwin-apps/shutdown/shutdown.8,v
retrieving revision 1.1
diff -u -p -r1.1 shutdown.8
--- shutdown.8	21 Mar 2013 11:58:00 -0000	1.1
+++ shutdown.8	25 Mar 2013 11:26:12 -0000
@@ -26,23 +26,29 @@ which specifies the time on the 24hr clo
 \fB\-f\fR, \fB\-\-force\fR
 Forces the execution. This can cause applications to lose data.
 .TP
-\fB\-s\fR, \fB\-\-shutdown\fR
+\fB\-h\fR, \fB\-\-halt\fR
 The system will shutdown and power off. This is the default action. The system must support the power-off feature, but most systems do.
 .TP
+\fB\-s\fR, \fB\-\-shutdown\fR
+Same as --halt. Here for compatibility with the Windows shutdown command line tool.
+.TP
 \fB\-r\fR, \fB\-\-reboot\fR
 The system will shutdown and then restart.
 .TP
-\fB\-h\fR, \fB\-\-hibernate\fR
+\fB\-b\fR, \fB\-\-hibernate\fR
 The system will suspend to disk. The system must support the hibernate feature.
 .TP
 \fB\-p\fR, \fB\-\-suspend\fR
 The system will suspend to RAM. This is also known as sleep or standby. The system must support the suspend feature.
 .TP
-\fB\-a\fR, \fB\-\-abort\fR
+\fB\-c\fR, \fB\-\-cancel\fR
 Aborts the execution of formerly started shutdown or reboot.
 .TP
+\fB\-a\fR, \fB\-\-abort\fR
+Same as --cancel. Here for compatibility with the Windows shutdown command line tool.
+.TP
 \fB\-x\fR, \fB\-\-exitex\fR
-Use ExitWindowsEx rather than InitateSystemShutdown.
+Use ExitWindowsEx rather than InitiateSystemShutdownEx.
 .TP
 \fB\-\-help\fR
 Display a help text and exit.
Index: shutdown.c
===================================================================
RCS file: /cvs/cygwin-apps/shutdown/shutdown.c,v
retrieving revision 1.4
diff -u -p -r1.4 shutdown.c
--- shutdown.c	21 Mar 2013 11:58:35 -0000	1.4
+++ shutdown.c	25 Mar 2013 11:26:12 -0000
@@ -63,12 +63,14 @@ usage_shutdown (void)
   printf ("Usage: %s [OPTION]... time\n", myname);
   printf ("Bring the system down.\n\n");
   printf ("  -f, --force      Forces the execution.\n");
+  printf ("  -h, --halt       The system will shutdown and power off (if supported)\n");
   printf ("  -s, --shutdown   The system will shutdown and power off (if supported)\n");
   printf ("  -r, --reboot     The system will reboot.\n");
-  printf ("  -h, --hibernate  The system will suspend to disk (if supported)\n");
+  printf ("  -b, --hibernate  The system will suspend to disk (if supported)\n");
   printf ("  -p, --suspend    The system will suspend to RAM (if supported)\n");
+  printf ("  -c, --cancel     Aborts execution of formerly started shutdown.\n");
   printf ("  -a, --abort      Aborts execution of formerly started shutdown.\n");
-  printf ("  -x, --exitex     Use ExitWindowsEx rather than InitateSystemShutdown.\n");
+  printf ("  -x, --exitex     Use ExitWindowsEx rather than InitiateSystemShutdownEx.\n");
   printf ("      --help       Display this help and exit.\n");
   printf ("      --version    Output version information and exit.\n");
   printf ("\n`time' is either the time in seconds or `+' and the time in minutes or a\n");
@@ -98,7 +100,7 @@ usage_reboot (void)
   }
 
   printf ("  -f, --force      Forces the execution.\n");
-  printf ("  -x, --exitex     Use ExitWindowsEx rather than InitateSystemShutdown.\n");
+  printf ("  -x, --exitex     Use ExitWindowsEx rather than InitiateSystemShutdownEx.\n");
   printf ("      --help       Display this help and exit.\n");
   printf ("      --version    Output version information and exit.\n");
   return 0;
@@ -166,18 +168,20 @@ parse_cmdline_shutdown(int argc, char **
 {
   struct option longopts[] = {
     {"abort", no_argument, NULL, 'a'},
+    {"cancel", no_argument, NULL, 'c'},
     {"exitex", no_argument, NULL, 'x'},
     {"force", no_argument, NULL, 'f'},
     {"shutdown", no_argument, NULL, 's'},
+    {"halt", no_argument, NULL, 'h'},
     {"reboot", no_argument, NULL, 'r'},
-    {"hibernate", no_argument, NULL, 'h'},
+    {"hibernate", no_argument, NULL, 'b'},
     {"suspend", no_argument, NULL, 'p'},
     {"help", no_argument, NULL, 'H'},
     {"version", no_argument, NULL, 'v'},
     {0, no_argument, NULL, 0}
   };
 
-  char opts[] = "axfsrhp";
+  char opts[] = "acxfshrbp";
   int c;
   char *arg, *endptr;
 
@@ -188,18 +192,20 @@ parse_cmdline_shutdown(int argc, char **
 	force = EWX_FORCE;
 	break;
       case 's':
+      case 'h':
         action = EWX_POWEROFF;
 	break;
       case 'r':
 	action = EWX_REBOOT;
 	break;
-      case 'h':
+      case 'b':
         action = HIBERNATE;
 	break;
       case 'p':
         action = SUSPEND;
 	break;
       case 'a':
+      case 'c':
 	action = ABORT;
 	break;
       case 'x':

[-- Attachment #4: Type: text/plain, Size: 218 bytes --]

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-20 11:41 Patch for shutdown Frank Fesevur
@ 2013-05-20 13:21 ` Jeffrey Altman
  2013-05-20 13:59   ` Fedin Pavel
  2013-05-20 20:32   ` Frank Fesevur
  2013-05-21  9:42 ` Corinna Vinschen
  1 sibling, 2 replies; 15+ messages in thread
From: Jeffrey Altman @ 2013-05-20 13:21 UTC (permalink / raw)
  To: cygwin

Frank,

I would be unhappy with this proposed change.   The Windows
"shutdown.exe" uses:

 'h' for hibernate
 'l' for logoff
 's' for shutdown and halt
 'r' for shutdown and restart
 'g' for shutdown and restart including registered applications

 'a' for abort shutdown

with the following modifiers:

 'f' for force
 'p' for 'no warning
 'e' document the reason
 'i' display GUI
 'm' specify target computer
 't' specify timeout
 'c' specify comment
 'd' shutdown reason code

I believe it is very important that the Cygwin shutdown not alter the
meaning of command line parameters such that they are different from the
Windows native version.   The various options are already too confusing
to remember.  Typing the right option value into wrong shell should not
result in the wrong behavior if we can help it.

Thank you.

Jeffrey Altman





On 5/20/2013 7:40 AM, Frank Fesevur wrote:
> Hi Corinna,
> 
> Since nobody made real objections about changing the short flag for
> hibernate from -h to -b to make room for -h for halt [1], I created a
> patch to change the shutdown program. The patch was created back in
> March, but with all your work being done on cygwin64, your holiday and
> myself being very busy with life it was held back.
> 
> There were no change to the shutdown CVS since then so here is that
> patch, including a changelog.
> 
> Note that I didn't change the programs version number in this patch. I
> don't know if changing an existing option would require/justify to
> change the version from 1.9 to 2.0 instead of simply to 1.10?
> 
> Regards,
> Frank
> 
> [1] http://cygwin.com/ml/cygwin/2013-03/msg00354.html
> 
> 
> 
> --
> Problem reports:       http://cygwin.com/problems.html
> FAQ:                   http://cygwin.com/faq/
> Documentation:         http://cygwin.com/docs.html
> Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
> 


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* RE: Patch for shutdown
  2013-05-20 13:21 ` Jeffrey Altman
@ 2013-05-20 13:59   ` Fedin Pavel
  2013-05-20 14:07     ` Earnie Boyd
                       ` (3 more replies)
  2013-05-20 20:32   ` Frank Fesevur
  1 sibling, 4 replies; 15+ messages in thread
From: Fedin Pavel @ 2013-05-20 13:59 UTC (permalink / raw)
  To: jaltman; +Cc: cygwin

> I would be unhappy with this proposed change.   The Windows
> "shutdown.exe" uses:
> 
>  'h' for hibernate
>  'l' for logoff
>  's' for shutdown and halt
>  'r' for shutdown and restart
>  'g' for shutdown and restart including registered applications
> 
>  'a' for abort shutdown
> 
> with the following modifiers:
> 
>  'f' for force
>  'p' for 'no warning
>  'e' document the reason
>  'i' display GUI
>  'm' specify target computer
>  't' specify timeout
>  'c' specify comment
>  'd' shutdown reason code
> 
> I believe it is very important that the Cygwin shutdown not alter the
> meaning of command line parameters such that they are different from
> the
> Windows native version.   The various options are already too confusing
> to remember.  Typing the right option value into wrong shell should not
> result in the wrong behavior if we can help it.

 At the other hand, IMHO, Windows .bat scripts are never run from wihin bash, and vice versa, UNIX .sh scripts are never run from within cmd.exe...
 And following the same logic we would need to teach our find.exe (already mentioned on this list) to understand Windows options instead of UNIX options...
 Even further, in terminal case, why have Cygwin at all ? It is different from Windows command line and this is confusing...

 Kind regards.


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-20 13:59   ` Fedin Pavel
@ 2013-05-20 14:07     ` Earnie Boyd
  2013-05-20 14:23     ` Jeffrey Altman
                       ` (2 subsequent siblings)
  3 siblings, 0 replies; 15+ messages in thread
From: Earnie Boyd @ 2013-05-20 14:07 UTC (permalink / raw)
  To: cygwin

On Mon, May 20, 2013 at 9:59 AM, Fedin Pavel wrote:
>
>  At the other hand, IMHO, Windows .bat scripts are never run from wihin bash, and vice versa, UNIX .sh scripts are never run from within cmd.exe...

I do this.

Create a null (native, i.e. non-cygwin) program named #! and put it in
PATH.  Then my scripts can do

~~~~~
#!/cmd /c

:: Dos batch commands
~~~~~

>  And following the same logic we would need to teach our find.exe (already mentioned on this list) to understand Windows options instead of UNIX options...

I think that would be a no go from the get go.

>  Even further, in terminal case, why have Cygwin at all ? It is different from Windows command line and this is confusing...
>

Oh, now I read the sarcasm. :/

-- 
Earnie
-- https://sites.google.com/site/earnieboyd

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-20 13:59   ` Fedin Pavel
  2013-05-20 14:07     ` Earnie Boyd
@ 2013-05-20 14:23     ` Jeffrey Altman
  2013-05-20 17:50     ` Andrey Repin
  2013-05-21 13:01     ` Buchbinder, Barry (NIH/NIAID) [E]
  3 siblings, 0 replies; 15+ messages in thread
From: Jeffrey Altman @ 2013-05-20 14:23 UTC (permalink / raw)
  To: cygwin

On 5/20/2013 9:59 AM, Fedin Pavel wrote:
> At the other hand, IMHO, Windows .bat scripts are never run from wihin
> bash, and vice versa, UNIX .sh scripts are never run from within
> cmd.exe... And following the same logic we would need to teach our
> find.exe (already mentioned on this list) to understand Windows
> options instead of UNIX options... Even further, in terminal case, why
> have Cygwin at all ? It is different from Windows command line and
> this is confusing... Kind regards. 

Unlike the 'find' command, the shutdown command can leave the machine in
a state which cannot be recovered without physical access.   That is the
difference as I see it and it is a significant difference.

Sincerely

Jeffrey Altman



--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-20 13:59   ` Fedin Pavel
  2013-05-20 14:07     ` Earnie Boyd
  2013-05-20 14:23     ` Jeffrey Altman
@ 2013-05-20 17:50     ` Andrey Repin
  2013-05-21 13:01     ` Buchbinder, Barry (NIH/NIAID) [E]
  3 siblings, 0 replies; 15+ messages in thread
From: Andrey Repin @ 2013-05-20 17:50 UTC (permalink / raw)
  To: Fedin Pavel, cygwin

Greetings, Fedin Pavel!

>  At the other hand, IMHO, Windows .bat scripts are never run from wihin
> bash, and vice versa, UNIX .sh scripts are never run from within cmd.exe...

That's a bold claim. If I run it down to underlying interpreter, I always run
all scripts from "cmd", be it .bat, .cmd, .btm, .sh or, say, .php.

>  And following the same logic we would need to teach our find.exe (already
> mentioned on this list) to understand Windows options instead of UNIX
> options...

Yeah, that would be ridiculous one.

>  Even further, in terminal case, why have Cygwin at all ? It is different
> from Windows command line and this is confusing... 

Heh.

Anyway, real argument is that Cygwin is providing tools that mimic *NIX
environment. What you want to use in the end of the day is up to you.
Supposedly, you're fully aware of the consequences.


--
WBR,
Andrey Repin (anrdaemon@freemail.ru) 20.05.2013, <21:37>

Sorry for my terrible english...


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-20 13:21 ` Jeffrey Altman
  2013-05-20 13:59   ` Fedin Pavel
@ 2013-05-20 20:32   ` Frank Fesevur
  1 sibling, 0 replies; 15+ messages in thread
From: Frank Fesevur @ 2013-05-20 20:32 UTC (permalink / raw)
  To: cygwin

2013/5/20 Jeffrey Altman:
> I believe it is very important that the Cygwin shutdown not alter the
> meaning of command line parameters such that they are different from the
> Windows native version.   The various options are already too confusing
> to remember.  Typing the right option value into wrong shell should not
> result in the wrong behavior if we can help it.

But why do you install the cygwin shutdown package when you want to
use the native windows shutdown command line options? The windows
shutdown.exe work just fine when it is run from cygwin (mintty, rxvt,
ssh). When you de-install the cygwin package, you can use the windows
shutdown command.

The cygwin shutdown tries to give a POSIX/Linux-like shutdown
experience and therefore should mimic the common Linux command line
options as good as possible. That's the reason I install the shutdown
package. In cygwin I type "ls", "less", etc. It feels like Linux, so I
want the shutdown to feel like Linux as well and use -h for halt just
like on any real Linux system.

And remember that the cygwin shutdown has long options and the "halt",
"reboot" and "hibernate" commands.

Regards,
Frank

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-20 11:41 Patch for shutdown Frank Fesevur
  2013-05-20 13:21 ` Jeffrey Altman
@ 2013-05-21  9:42 ` Corinna Vinschen
  2013-05-21 15:20   ` Frank Fesevur
  1 sibling, 1 reply; 15+ messages in thread
From: Corinna Vinschen @ 2013-05-21  9:42 UTC (permalink / raw)
  To: cygwin

On May 20 13:40, Frank Fesevur wrote:
> Hi Corinna,
> 
> Since nobody made real objections about changing the short flag for
> hibernate from -h to -b to make room for -h for halt [1], I created a
> patch to change the shutdown program. The patch was created back in
> March, but with all your work being done on cygwin64, your holiday and
> myself being very busy with life it was held back.

I'm a bit reluctant as of the backward compatibility breakage.  What
if somebody uses shutdown in a script?  You know, starting a backup
in the evening and the last action of the script is to hibernate the
machine or something like that...

> There were no change to the shutdown CVS since then so here is that
> patch, including a changelog.
> 
> Note that I didn't change the programs version number in this patch. I
> don't know if changing an existing option would require/justify to
> change the version from 1.9 to 2.0 instead of simply to 1.10?

1.10 is fine I think.  Say, do you want to take over shutdown
maintainership?  It's an easy enough package to start with, it
builds fine on 32 and 64 bit, and you get the cygport configuration
"for free".  What do you say?


Corinna

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

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* RE: Patch for shutdown
  2013-05-20 13:59   ` Fedin Pavel
                       ` (2 preceding siblings ...)
  2013-05-20 17:50     ` Andrey Repin
@ 2013-05-21 13:01     ` Buchbinder, Barry (NIH/NIAID) [E]
  3 siblings, 0 replies; 15+ messages in thread
From: Buchbinder, Barry (NIH/NIAID) [E] @ 2013-05-21 13:01 UTC (permalink / raw)
  To: cygwin

Fedin Pavel sent the following at Monday, May 20, 2013 10:00 AM
> And following the same logic we would need to teach our find.exe (already mentioned on this list) to understand Windows options instead of UNIX options...

FYI, Windows find = grep

c:\> find /?
Searches for a text string in a file or files.

FIND [/V] [/C] [/N] [/I] [/OFF[LINE]] "string" [[drive:][path]filename[ ...]]

  /V         Displays all lines NOT containing the specified string.
  /C         Displays only the count of lines containing the string.
  /N         Displays line numbers with the displayed lines.
  /I         Ignores the case of characters when searching for the string.
  /OFF[LINE] Do not skip files with offline attribute set.
  "string"   Specifies the text string to find.
  [drive:][path]filename
             Specifies a file or files to search.

If a path is not specified, FIND searches the text typed at the prompt
or piped from another command.

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

* Re: Patch for shutdown
  2013-05-21  9:42 ` Corinna Vinschen
@ 2013-05-21 15:20   ` Frank Fesevur
  2013-05-21 16:05     ` Corinna Vinschen
  0 siblings, 1 reply; 15+ messages in thread
From: Frank Fesevur @ 2013-05-21 15:20 UTC (permalink / raw)
  To: cygwin

2013/5/21 Corinna Vinschen:
> I'm a bit reluctant as of the backward compatibility breakage.  What
> if somebody uses shutdown in a script?  You know, starting a backup
> in the evening and the last action of the script is to hibernate the
> machine or something like that...

That's why I sent my email in March in the first place. But I really
think having -h for halt is better for a POSIX/Linux-like shutdown.
And "shutdown -h now" will now halt the machine instead of hibernate.
Although I understand they definitely are not the same, it won't leave
the machine turned on. It will be in a "form of off".

A proper announcement is required. The backward compatibility breakage
was the reason I thought about 2.x vs 1.x.

> Say, do you want to take over shutdown
> maintainership?  It's an easy enough package to start with, it
> builds fine on 32 and 64 bit, and you get the cygport configuration
> "for free".  What do you say?

Anything "for free" must be good ;-)

I see the cygport file is in src archive. It has been very long since
I have played with cygport.

And I have not looked at cygwin64, although I use Win7-64. I had no
real need for cygwin64 so far, so I wanted to wait for it to be
officially released. I just ran the setup64.exe found at a nearby
mirror, installed cygport and gcc/g++ and shutdown compiles under 32
and 64 bit.

So back to the question... why not take over? I will read
http://cygwin.com/setup.html thoroughly, create the new packages and
send the ITA any time soon.

You keep the sources in CVS right now. But when I take over, I assume
it is ok I put them at my github.

Regards,
Frank

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-21 15:20   ` Frank Fesevur
@ 2013-05-21 16:05     ` Corinna Vinschen
  2013-05-21 20:00       ` Frank Fesevur
  0 siblings, 1 reply; 15+ messages in thread
From: Corinna Vinschen @ 2013-05-21 16:05 UTC (permalink / raw)
  To: cygwin

On May 21 17:20, Frank Fesevur wrote:
> 2013/5/21 Corinna Vinschen:
> > I'm a bit reluctant as of the backward compatibility breakage.  What
> > if somebody uses shutdown in a script?  You know, starting a backup
> > in the evening and the last action of the script is to hibernate the
> > machine or something like that...
> 
> That's why I sent my email in March in the first place. But I really
> think having -h for halt is better for a POSIX/Linux-like shutdown.
> And "shutdown -h now" will now halt the machine instead of hibernate.
> Although I understand they definitely are not the same, it won't leave
> the machine turned on. It will be in a "form of off".
> 
> A proper announcement is required. The backward compatibility breakage
> was the reason I thought about 2.x vs 1.x.
> 
> > Say, do you want to take over shutdown
> > maintainership?  It's an easy enough package to start with, it
> > builds fine on 32 and 64 bit, and you get the cygport configuration
> > "for free".  What do you say?
> 
> Anything "for free" must be good ;-)
> 
> I see the cygport file is in src archive. It has been very long since
> I have played with cygport.
> 
> And I have not looked at cygwin64, although I use Win7-64. I had no
> real need for cygwin64 so far, so I wanted to wait for it to be
> officially released. I just ran the setup64.exe found at a nearby
> mirror, installed cygport and gcc/g++ and shutdown compiles under 32
> and 64 bit.
> 
> So back to the question... why not take over? I will read
> http://cygwin.com/setup.html thoroughly, create the new packages and
> send the ITA any time soon.

Cool, thanks!

> You keep the sources in CVS right now. But when I take over, I assume
> it is ok I put them at my github.

Uhm... I would prefer if we had the cygwin-specific tools in a single
repository, if it's not asked too much.  I'm sure we could add git
access as well, if it's not already available.


Corinna

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

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-21 16:05     ` Corinna Vinschen
@ 2013-05-21 20:00       ` Frank Fesevur
  2013-05-22  8:18         ` Corinna Vinschen
  0 siblings, 1 reply; 15+ messages in thread
From: Frank Fesevur @ 2013-05-21 20:00 UTC (permalink / raw)
  To: cygwin

2013/5/21 Corinna Vinschen:
> Uhm... I would prefer if we had the cygwin-specific tools in a single
> repository, if it's not asked too much.  I'm sure we could add git
> access as well, if it's not already available.

I can continue to use CVS, no problem. No need to use git, just for
the sake of git ;-)

But I guess then I'll need write access or you must still review and
commit the patches I write.

Regards,
Frank

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-21 20:00       ` Frank Fesevur
@ 2013-05-22  8:18         ` Corinna Vinschen
  2013-06-06 12:22           ` Frank Fesevur
  0 siblings, 1 reply; 15+ messages in thread
From: Corinna Vinschen @ 2013-05-22  8:18 UTC (permalink / raw)
  To: cygwin

On May 21 22:00, Frank Fesevur wrote:
> 2013/5/21 Corinna Vinschen:
> > Uhm... I would prefer if we had the cygwin-specific tools in a single
> > repository, if it's not asked too much.  I'm sure we could add git
> > access as well, if it's not already available.
> 
> I can continue to use CVS, no problem. No need to use git, just for
> the sake of git ;-)
> 
> But I guess then I'll need write access or you must still review and
> commit the patches I write.

Write access is easy:

http://sourceware.org/cgi-bin/pdw/ps_form.cgi

Use "cygwin-apps" as project,  corinna at vinschen dot de as "email
address of person who approved request".

After that, checking out the project is done using 

  $ cvs -d :ext:<your login>@sourceware.org:/cvs/cygwin-apps co shutdown


Thanks,
Corinna

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

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-05-22  8:18         ` Corinna Vinschen
@ 2013-06-06 12:22           ` Frank Fesevur
  2013-06-06 14:29             ` Corinna Vinschen
  0 siblings, 1 reply; 15+ messages in thread
From: Frank Fesevur @ 2013-06-06 12:22 UTC (permalink / raw)
  To: cygwin

2013/5/22 Corinna Vinschen:
> After that, checking out the project is done using
>
>   $ cvs -d :ext:<your login>@sourceware.org:/cvs/cygwin-apps co shutdown

The code is in CVS.

Since this is my first package and it uses the CVS I have a question
about the next steps.

If I understand things correctly, these are the steps needed:
- add tag "rel1-10" to CVS
- use cygport to generate the bin packages
  - "cygport --32 shutdown.cygport download prep compile package"
  - "cygport --64 shutdown.cygport download prep compile package"
- I assume these commands will also generate a src package

From there I follow the normal steps (ITA, RFU and announcement) as
described on http://cygwin.com/setup.html

Did I miss anything?

Regards,
Frank

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

* Re: Patch for shutdown
  2013-06-06 12:22           ` Frank Fesevur
@ 2013-06-06 14:29             ` Corinna Vinschen
  0 siblings, 0 replies; 15+ messages in thread
From: Corinna Vinschen @ 2013-06-06 14:29 UTC (permalink / raw)
  To: cygwin

On Jun  6 14:21, Frank Fesevur wrote:
> 2013/5/22 Corinna Vinschen:
> > After that, checking out the project is done using
> >
> >   $ cvs -d :ext:<your login>@sourceware.org:/cvs/cygwin-apps co shutdown
> 
> The code is in CVS.
> 
> Since this is my first package and it uses the CVS I have a question
> about the next steps.
> 
> If I understand things correctly, these are the steps needed:
> - add tag "rel1-10" to CVS
> - use cygport to generate the bin packages
>   - "cygport --32 shutdown.cygport download prep compile package"
>   - "cygport --64 shutdown.cygport download prep compile package"
> - I assume these commands will also generate a src package
> 
> >From there I follow the normal steps (ITA, RFU and announcement) as
> described on http://cygwin.com/setup.html
> 
> Did I miss anything?

Looks good to me.  Btw., maintainer issues are usually discussed on
the cygwin-apps list.  Feel free to discuss any further issue there.


Thanks,
Corinna

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

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

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

end of thread, other threads:[~2013-06-06 14:29 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-20 11:41 Patch for shutdown Frank Fesevur
2013-05-20 13:21 ` Jeffrey Altman
2013-05-20 13:59   ` Fedin Pavel
2013-05-20 14:07     ` Earnie Boyd
2013-05-20 14:23     ` Jeffrey Altman
2013-05-20 17:50     ` Andrey Repin
2013-05-21 13:01     ` Buchbinder, Barry (NIH/NIAID) [E]
2013-05-20 20:32   ` Frank Fesevur
2013-05-21  9:42 ` Corinna Vinschen
2013-05-21 15:20   ` Frank Fesevur
2013-05-21 16:05     ` Corinna Vinschen
2013-05-21 20:00       ` Frank Fesevur
2013-05-22  8:18         ` Corinna Vinschen
2013-06-06 12:22           ` Frank Fesevur
2013-06-06 14: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).