[Rubycocoa-devel 968] Re: rubycocoa command

Back to archive index

Eloy Duran eloy.****@gmail*****
Sun Jun 10 05:45:36 JST 2007


Hey,

I have a little patch that fixes some text errors :)
Another thing is that the apply_template method doesn't seem to work
on the InfoPlist.strings file, so when you run the app the name is
still <<PROJECTNAME>>.

Index: rubycocoa
===================================================================
--- rubycocoa   (revision 1826)
+++ rubycocoa   (working copy)
@@ -83,15 +83,15 @@
       parser.banner  = <<-EOB.gsub(/^ {8}/, "")
         Usage: rubycocoa <subcommand> [options] <files>

-        `rubycocoa' command is a part of RubyCocoa.
-        This is for creating new application skelton.
+        The 'rubycocoa' command is a part of RubyCocoa.
+        It's a tool for creating new application skeltons.

         Intro:
-                You can make RubyCocoa application skelton with `new'
subcommand.
+                You can create a RubyCocoa application skelton with
the 'new' subcommand:

-                    rubycocoa new "New RubyCocoa Application"
+                    $ rubycocoa new "New RubyCocoa Application"

-                After selecting template, rubycocoa make applicatoin
dir in current.
+                After selecting the template, 'rubycocoa' will create
the application directory in the current directory.
       EOB

       parser.separator ""
@@ -130,7 +130,7 @@

   def cmd_new
     appname, = @argv
-    abort "Application Name must be required" if appname.empty?
+    abort "Application Name is required" if appname.nil? || appname.empty?

     templates = Pathname.glob("#{ENV["HOME"]}/.rubycocoa/templates/*")
     templates.concat Pathname.glob("/Library/Application
Support/Apple/Developer Tools/Project
Templates/Application/Cocoa-Ruby*")
@@ -140,14 +140,14 @@
     print "Select Template> "
     num = $stdin.gets.chomp
     num = "0" if num.empty?
-    abort "Canceled since inputed not a number" unless num =~ /^\d/
+    abort "Canceled because the input was not a number" unless num =~ /^\d/

     num = num.to_i
     template = templates[num]
     puts "Creating `#{appname}' using `#{template.basename}'..."

     dest = Pathname.new(appname)
-    abort "#{appname} is exists. Cancel." if dest.exist?
+    abort "#{appname} already exists. Exiting." if dest.exist?
     cp_r template, dest
     Pathname.glob(dest + "*Cocoa*App*") do |f|
       f.rename(f.parent + f.basename.to_s.sub(/Cocoa(?:Doc)?App/, appname))

Cheers,
Eloy

On 6/9/07, Eloy Duran <eloy.****@gmail*****> wrote:
> I just quickly skimmed the sources and it's looking great!
> 2 very little observations:
> - I think it's better if we keep all the sources free from tabs, but
> instead use 2 spaces per tab (I noticed that 2 other source files in
> the framework/src/ruby/osx/objc dir have some tabs in them too). But
> this is my personal opinion.... :)
> - I would like to see the tests inside the source_root/test dir, so it
> will also be run by: $ ruby install.rb test
>
> Now I'm really going to play with it :)
>
> Thanks!
>
> On 6/9/07, SATOH Hiroh <cho45****@lowre*****> wrote:
> > I see. Thank you :)
> >
> > Eloy Duran さんは書きました:
> > > Hey Hiroh-san,
> > >
> > > I would say those places seem best,
> > > but it's trivial to move them later on so go for it. :)
> > >
> > > Cheers,
> > > Eloy
> > >
> >
> >
> > --
> > cho45, (さとうひろお) SATOH, Hiroh
> > http://lowreal.net/
> >
> > _______________________________________________
> > Rubycocoa-devel mailing list
> > Rubyc****@lists*****
> > http://lists.sourceforge.jp/mailman/listinfo/rubycocoa-devel
> >
>




More information about the Rubycocoa-devel mailing list
Back to archive index