Re: [ros-dev] [ros-diffs] [gedmurphy] 23696: add an 'uptime' command. Available in either Windows or Linux output. (defaulted to Windows)

[Alex Ionescu]

gedmurphy(a)svn.reactos.org wrote:
>   Author: gedmurphy
>  Date: Fri Aug 25 04:33:27 2006
>  New Revision: 23696
>  
>  URL: http://svn.reactos.org/svn/reactos?rev=23696&view=rev
>  Log:
>  add an 'uptime' command.
>  Available in either Windows or Linux output. (defaulted to Windows)
>   
Please don't add 3rd-party utilities to the tree. Such applications need
to go in ros-apps. Base is only for BASE applications. "uptime" isn't.

-- 
Best regards,
Alex Ionescu
Project Lead, TinyKRNL
Kernel-Mode Software Design Engineer, ReactOS

[Brandon Turner]

Alex Ionescu wrote:
>   gedmurphy(a)svn.reactos.org wrote:
>    
> >   Author: gedmurphy
> >  Date: Fri Aug 25 04:33:27 2006
> >  New Revision: 23696
> >
> >  URL: http://svn.reactos.org/svn/reactos?rev=23696&view=rev
> >  Log:
> >  add an 'uptime' command.
> >  Available in either Windows or Linux output. (defaulted to Windows)
> >
> >       
>  Please don't add 3rd-party utilities to the tree. Such applications need
>  to go in ros-apps. Base is only for BASE applications. "uptime" isn't.
>
>     
What I always thought was "base applications" were apps there were 
included in windows by default. And rosapps for was for any other apps 
we wanted to include.  "Uptime" would then fall under rosapps, however 
if this was included as a "systeminfo" replacement it would go into base 
applications.  Is this what other people thought as well?

-- 
Brandon Turner
ReactOS Release Engineer
Blog: http://www.amateurgramming.com

[Alex Ionescu]

Brandon Turner wrote:
>   Alex Ionescu wrote:
>  
> >  gedmurphy(a)svn.reactos.org wrote:
> >   
> >
> > >  Author: gedmurphy
> > > Date: Fri Aug 25 04:33:27 2006
> > > New Revision: 23696
> > >
> > > URL: http://svn.reactos.org/svn/reactos?rev=23696&view=rev
> > > Log:
> > > add an 'uptime' command.
> > > Available in either Windows or Linux output. (defaulted to Windows)
> > >
> > >      
> > Please don't add 3rd-party utilities to the tree. Such applications need
> > to go in ros-apps. Base is only for BASE applications. "uptime" isn't.
> >
> >    
>  
>  What I always thought was "base applications" were apps there were 
>  included in windows by default. And rosapps for was for any other apps 
>  we wanted to include.  "Uptime" would then fall under rosapps, however 
>  if this was included as a "systeminfo" replacement it would go into base 
>  applications.  Is this what other people thought as well?
>   
Yes, this is what everyone but Ged thinks/knows.

-- 
Best regards,
Alex Ionescu
Project Lead, TinyKRNL
Kernel-Mode Software Design Engineer, ReactOS

[Brandon Turner]

Alex Ionescu wrote:
>   Brandon Turner wrote:
>    
> >   Alex Ionescu wrote:
> >
> >      
> > >   gedmurphy(a)svn.reactos.org wrote:
> > >   
> > >
> > >        
> > > >   Author: gedmurphy
> > > >  Date: Fri Aug 25 04:33:27 2006
> > > >  New Revision: 23696
> > > >
> > > >  URL: http://svn.reactos.org/svn/reactos?rev=23696&view=rev
> > > >  Log:
> > > >  add an 'uptime' command.
> > > >  Available in either Windows or Linux output. (defaulted to Windows)
> > > >
> > > >     
> > > >           
> > >  Please don't add 3rd-party utilities to the tree. Such applications
> > > need
> > >  to go in ros-apps. Base is only for BASE applications. "uptime" isn't.
> > >
> > >   
> > >         
> >  What I always thought was "base applications" were apps there
> > were 
> >  included in windows by default. And rosapps for was for any other apps 
> >  we wanted to include.  "Uptime" would then fall under rosapps, however 
> >  if this was included as a "systeminfo" replacement it would go into base 
> >  applications.  Is this what other people thought as well?
> >
> >       
>  Yes, this is what everyone but Ged thinks/knows.
>
>     
Someone has their sassy pants on tonight. ;)

-- 
Brandon Turner
ReactOS Release Engineer
Blog: http://www.amateurgramming.com

[James Tabor]

Brandon Turner:
> >  
> > Yes, this is what everyone but Ged thinks/knows.
> >
> >    
>  
>  Someone has their sassy pants on tonight. ;)
>   
beside all that! Its broke,

make[1]: Entering directory `/home/ros/reactos'
[CC]       base/applications/cmdutils/uptime/uptime.c
base/applications/cmdutils/uptime/uptime.c: In function `main':
base/applications/cmdutils/uptime/uptime.c:28: warning: unsigned int format, different
type arg (arg2)
base/applications/cmdutils/uptime/uptime.c:28: warning: unsigned int format, different
type arg (arg3)
base/applications/cmdutils/uptime/uptime.c:28: warning: unsigned int format, different
type arg (arg4)
base/applications/cmdutils/uptime/uptime.c:28: warning: unknown conversion type character
`I' in format
base/applications/cmdutils/uptime/uptime.c:28: warning: too many arguments for format
make[1]: *** [obj-i386/base/applications/cmdutils/uptime/uptime.o] Error 1
make[1]: Leaving directory `/home/ros/reactos'
make: *** [depends] Error 2