sketchucation logo sketchucation
    • Login
    ⚠️ Attention | Having issues with Sketchucation Tools 5? Report Here

    Aborting a .rbz install

    Scheduled Pinned Locked Moved Developers' Forum
    16 Posts 4 Posters 477 Views 4 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • TIGT Offline
      TIG Moderator
      last edited by

      IF you put the 'IS_MAC' test at the start of the Plugins-folder-level script that is 'auto-loading' as the file installs [perhaps outside of the main module] and then you add 'the-message' + then delete itself AND all other just-added subfolders' files, AND then delete those empty-subfolders, AND THEN 'return nil' [before any other other code in the 'loader' is run] then NO other code should load at all ... The subfolders of support files will have been installed and then immediately deleted if !IS_MAC ???

      TIG

      1 Reply Last reply Reply Quote 0
      • D Offline
        driven
        last edited by

        On the mac this removes everything, except my main targets
        i.e the primary folder is retained containing a [dot] folder containing a mac binary file
        I get Error: #<Errno::ENOTEMPTY:

        The question is, would Windows 'protect' these file types or would they be deleted as well?

        if( ( RUBY_PLATFORM =~ /darwin/i) )
        UI.messagebox("\n This code is for Mac only! \n
        It will not run on Windows or Linux \n
        Loading will now abort \n
        Contact the Author if you wish to port the source \n
        ")
          path = File.expand_path(__FILE__)
          dirname = path.sub(".rb", "")
          Dir[(dirname + "{**/**/**/*")].sort.reverse.each do | p |
           if File.directory? p then
                    Dir.rmdir p
                elsif
                    File.exists? p then
                    File.delete p
                else
             end
            end
             File.delete path
             Dir.rmdir dirname
             return nil
          end
        

        john

        learn from the mistakes of others, you may not live long enough to make them all yourself...

        1 Reply Last reply Reply Quote 0
        • TIGT Offline
          TIG Moderator
          last edited by

          On a PC [and MAC?] this deletes a folder, even with deeply nested folders/files

          def self.delete_folder(d)
          		return nil unless File.directory?(d)
          		Dir.entries(d).each{|f|
          			next if f=='.' || f=='..'
          			x=File.join(d, f)
          			if File.directory?(x)
          				self.delete_folder(x)
          			else ### file
          				File.delete(x)
          			end
          		}
          		begin
          			Dir.delete(d)
          		rescue ### fails if it's not empty!
          			self.delete_folder(d)
          		end
          end
          

          Usage: self.delete_folder(full_path_to_folder_to_delete)
          You can't delete the . or .. entries in a folder, BUT you can delete an empty folder that only contains those two entries... It has the 'self.' only because I had this example in a module...

          TIG

          1 Reply Last reply Reply Quote 0
          • D Offline
            driven
            last edited by

            cheers,
            I got it working by 'naming' the difficult ones
            but I'll have a play with your version tomorrow...

            I think it's worth not loading the binary into a PC, even if it won't hurt...

            your up late

            john

            learn from the mistakes of others, you may not live long enough to make them all yourself...

            1 Reply Last reply Reply Quote 0
            • danielbowringD Offline
              danielbowring
              last edited by

              Deleting it seems like overkill, you could employ something like this instead

              
              module AuthorModule
                  def self.is_mac?()
                      # ...
                  end
              
                  module PluginModule
              
                  end if is_mac?
              
                  mac_only_warning unless is_mac?
              end
              
              

              Or in the extension file

              
              if is_mac?
                  extension = # ...
              else
                  mac_only_warning
              end
              
              
              1 Reply Last reply Reply Quote 0
              • D Offline
                driven
                last edited by

                having 'only works on mac' should be enough, but never is.

                I'm just wanting to avoid the 'this isn't working' posts from 'PC' users.

                Beside anything else, it would be trivial for SU to pre-screen for files marked as os dependant.

                john

                learn from the mistakes of others, you may not live long enough to make them all yourself...

                1 Reply Last reply Reply Quote 0
                • danielbowringD Offline
                  danielbowring
                  last edited by

                  @driven said:

                  having 'only works on mac' should be enough, but never is.

                  I'm just wanting to avoid the 'this isn't working' posts from 'PC' users.

                  Beside anything else, it would be trivial for SU to pre-screen for files marked as os dependant.

                  john

                  Note the if is_mac? at the end of PluginModule. That will prevent the plugin from "loading" anything more. Same if you just don't register the SketchupExtension. You could also write a setting to prevent it re-showing the message using Sketchup.write_default / Sketchup.read_default

                  Edit: Demo

                  
                  module MyModule
                      puts "This section will never be executed!"
                  end if false # because of this!
                  
                  puts "But this will!"
                  
                  puts defined?(MyModule)
                  
                  

                  Gives

                  
                  But this will!
                  nil
                  
                  
                  1 Reply Last reply Reply Quote 0
                  • TIGT Offline
                    TIG Moderator
                    last edited by

                    BUT we are trying to stop an RBZ install of a plugin onto an incompatible OS [a relatively rare occurrence?]...
                    So we need to check the OS early on, then tell the user what we are doing and delete the files/subfolders of files that ship in the RBZ [including itself!] ans stop loading anything at all.
                    It pointless having a plugin installed into an OS where it won't work...

                    TIG

                    1 Reply Last reply Reply Quote 0
                    • D Offline
                      driven
                      last edited by

                      @aerilius said:

                      .. or maybe the plugin folder is synced with another system on which the plugin is supposed to be used).

                      Excellent point I hadn't considered.

                      I was already leaning towards adding the [ is_mac? ] to the menu/toolbar loader, which which will prevent unusable menu items when on the wrong platform...

                      As my plugin's 'logic' is dynamically loaded into SU by either of these items being 'clicked', then it will also remain inert without any additional screening.

                      This, in addition to setting the extension i.e.
                      Sketchup.register_extension( add_icon_to_mac_skp, false) unless is_mac?
                      should accommodate a shared Plugin folders.

                      further thoughts?

                      john

                      learn from the mistakes of others, you may not live long enough to make them all yourself...

                      1 Reply Last reply Reply Quote 0
                      • TIGT Offline
                        TIG Moderator
                        last edited by

                        If you have mixed OSs on a network which share plugins you are making some issues for yourself... 😕
                        Only a few plugins ship in PC and MAC 'version's anyway, and handle OS differences within their methods...
                        So just allow the plugin to install on any OS...
                        but as suggested... test in the main 'loader' for the appropriate OS and only set up its Extension if there's an OS match...
                        That way you could have it installed on a shared folder that resides on a PC, that can never actually load it, but it can successfully load from that shared folder if it's called by a MAC etc...
                        😒

                        TIG

                        1 Reply Last reply Reply Quote 0
                        • A Offline
                          Aerilius
                          last edited by

                          Are there no risks when messing around with file deletions?
                          [If such a method existed] I would prefer to use corresponding API methods for (un)installation, or more generally put:
                          The same method for uninstallation that was used for installation (by hand, API or EWH).

                          What is the problem of keeping the plugin and just telling the user about the issue. This gives the clearest explanation and the least surprise. Also the user could maybe have reasons to keep the plugin installed without using it actively (like only inspecting how it works, or maybe the plugin folder is synced with another system on which the plugin is supposed to be used). Suddenly disappearing files would rather make a user worried and would blur the line between good software and malware.

                          Edit: Another thing is: Using ruby to delete files circumvents the trash. If we let the user manage "his" files (that the user installed), there is the chance to restore them.

                          1 Reply Last reply Reply Quote 0
                          • D Offline
                            driven
                            last edited by

                            one last idea
                            what happens on Windows if I add this to the top of each script

                            windows use = nil
                            
                            

                            If done correctly on a mac it returns nil and the ruby continues.

                            @unknownuser said:

                            you cantype non-breaking spaces directly on OS X with Option+Space.

                            I'm under the impression Windows will see it as a space and error out
                            Error: #<NoMethodError: undefined methodwindows' for #Object:0x56b89ec>`

                            If so, I can add it and even if the file/folders are there, nothing will run unless your on a mac...

                            pilfered from here:
                            http://www.rubyinside.com/the-split-is-not-enough-whitespace-shenigans-for-rubyists-5980.html

                            learn from the mistakes of others, you may not live long enough to make them all yourself...

                            1 Reply Last reply Reply Quote 0
                            • danielbowringD Offline
                              danielbowring
                              last edited by

                              @driven said:

                              I'm under the impression Windows will see it as a space and error out
                              Error: #<NoMethodError: undefined methodwindows' for #Object:0x56b89ec>`

                              Depends what encoding you save it in.

                              Western (Windows 1252) gives this:

                              
                              Error; #<SyntaxError; (eval);5231;in `load'; path/to/nbsp.rb;1; Invalid char `\240' in expression>
                              
                              

                              UTF-8, however, will load and work fine

                              
                              windows use = "o.~"
                              puts "Success #{windows use.inspect}"
                              
                              # ->
                              
                              load 'path/to/nbsp.rb'
                              Success "o.~"
                              true
                              
                              
                              

                              Note: Browsers will replace the character in question (160, "non-breaking space") with a regular space (32, "space"), but I did correct for this for this test.

                              1 Reply Last reply Reply Quote 0
                              • 1 / 1
                              • First post
                                Last post
                              Buy SketchPlus
                              Buy SUbD
                              Buy WrapR
                              Buy eBook
                              Buy Modelur
                              Buy Vertex Tools
                              Buy SketchCuisine
                              Buy FormFonts

                              Advertisement