search

cviebrock / sequel-pro-laravel-export

A Sequel Pro / Sequel Ace bundle to generate Laravel migration files from existing tables.
MIT License
921 stars 56 forks source link

Support for Sequal Ace? #35

Closed shiroamada closed 3 years ago

shiroamada commented 3 years ago

Planning to make a version support Sequal Ace?

When I run the Export to Laravel Migration. Below Error Message Prompt:

Error for “Export To Laravel Migration”: ./parse.sh: /usr/bin/env: bad interpreter: Operation not permitted

Thanks for your contribution.

cviebrock commented 3 years ago

To be honest, this is the first I've heard of Sequel Ace!

Their documentation is a bit sparse right now, but if I can get more info on the project, I'll try and port my bundle over.

However, that error must be in reference to the hash-bang line on the parse.sh file. What version of OS X are you running? And do you have a bash shell? echo $0 in Terminal should tell you what shell you are currently running, and check if /bin/bash exists.

shiroamada commented 3 years ago

Sorry for forgot to include OS version. MacOS Catalina 10.15.7 (19H2)

the current spBundle work well in Sequal Pro, but not working in Sequal Ace.

» echo $0                                                                                                                                               
-zsh

The bundles location is in this location /Users/shiroamada/Library/Containers/com.sequel-ace.sequel-ace/Data/Library/Application Support/Sequel Ace/Bundles/ExportToLaravelMigration.spBundle

cviebrock commented 3 years ago

Try:

xattr -l "/Users/shiroamada/Library/Containers/com.sequel-ace.sequel-ace/Data/Library/Application Support/Sequel Ace/Bundles/ExportToLaravelMigration.spBundle/parse.sh"

and

xattr -d com.apple.quarantine "/Users/shiroamada/Library/Containers/com.sequel-ace.sequel-ace/Data/Library/Application Support/Sequel Ace/Bundles/ExportToLaravelMigration.spBundle/parse.sh"

if you see the quarantine attribute.

shiroamada commented 3 years ago

Yes, for the first command. After exec, it feedback com.apple.quarantine: 0081;5fb66eaf;Chrome;B460A8E3-4EF2-40F5-87BD-913BDB125BDB

but second one does not have feedback.

However, after execute both command still having the same error

Error for “Export To Laravel Migration”:
./parse.sh: /usr/bin/env: bad interpreter: Operation not permitted

When I run ll

» ll "/Users/shiroamada/Library/Containers/com.sequel-ace.sequel-ace/Data/Library/Application Support/Sequel Ace/Bundles/ExportToLaravelMigration.spBundle/" 
total 80
-rwxrwxr-x@ 1 shiroamada  staff    19K Mar 10  2020 MigrationParser.php
-rwxrwxr-x@ 1 shiroamada  staff   935B Mar 10  2020 command.plist
-rwxrwxr-x@ 1 shiroamada  staff   788B Mar 10  2020 create.stub
-rwxrwxr-x@ 1 shiroamada  staff   307B Mar 10  2020 parse.php
-rwxr-xr-x  1 shiroamada  staff   5.6K Mar 10  2020 parse.sh

all is chmod +x

cviebrock commented 3 years ago

Can you run xattr -d com.apple.quarantine against all the files in the bundle directory? From what I've researched, that error has something to do with application signing. I'm guessing Sequel Ace is a signed app now, so dropping third-party bundle files is going to cause some issues.

cviebrock commented 3 years ago

@shiroamada Can you try downloading the Sequel Ace release from here: https://github.com/cviebrock/sequel-pro-laravel-export/releases/tag/2.0.0

It turns out there are some internal changes between Sequel Pro and Ace that required me to make a separate bundle for each.

Note: You may need to wait until Sequel Ace releases version 3.0 in order for the .sabundle file to work. In the meantime, you could download the .spbundle file and look line 14 in parse.sh:

    open "sequelpro://$SP_PROCESS_ID@passToDoc/ExecuteQuery"

Just replace sequelpro:// with sequelace://, then try installing the bundle.

Let me know how it works, please!

shiroamada commented 3 years ago

I downloaded the Sequal Ace Beta (beta/3.0.0-3001) to try it out. Still no chance get it work. image

I also try added the permission for /Users/shiroamada/Library/Containers/com.sequel-ace.sequel-ace-beta/Data/Library/Application Support/Sequel Ace Beta/Bundles/ExportToLaravelMigration.saBundle image

cviebrock commented 3 years ago

Try re-downloading the bundle from https://github.com/cviebrock/sequel-pro-laravel-export/releases/tag/2.0.0

I removed the old tag, made a fix, and re-tagged and released. I've tested it locally with the beta of SA 3.0.0 and it seems to be fine now.

You'll also need to give SA permissions to write to the Desktop folder. See the new docs for this package for a screenshot of what I mean.

hasanakhtar commented 3 years ago

Downloaded the https://github.com/cviebrock/sequel-pro-laravel-export/releases/tag/2.0.0 but unzipping does not have a file to run, instead has a folder ExportToLaravelMigration.saBundle.

Not able to install for Ace.

On Mac OS MOjave 10.14.6

cviebrock commented 3 years ago

Are you using the Sequel Ace Beta? If so, just dragging that .sabundle file onto the application should install it.

hasanakhtar commented 3 years ago

There is no file but a whole folder with this name "ExportToLaravelMigration.saBundle"

Using Sequel Ace version 2.3.2

On Thu, Dec 3, 2020 at 7:14 PM Colin Viebrock notifications@github.com wrote:

Are you using the Sequel Ace Beta? If so, just dragging that .sabundle file onto the application should install it.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cviebrock/sequel-pro-laravel-export/issues/35#issuecomment-738002547, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMLDSB57755XQOJWKFGGD7TSS6I5FANCNFSM4T3UD2OQ .

-- Thanks and Regards,

Hasan Akhtar Naqvi

cviebrock commented 3 years ago

@hasanakhtar My bundle will only work with Sequel Ace 3.0.0, as they have changed a few of the internals on how it works with third-party bundles.

You can download the beta version of 3.0.0 from here. I would assume that the release version will be available from the App Store shortly after they finalize their changes.

As for this:

Downloaded the https://github.com/cviebrock/sequel-pro-laravel-export/releases/tag/2.0.0 but unzipping does not have a file to run, instead has a folder ExportToLaravelMigration.saBundle.

Try downloading it again from that link. There was a mistake in my release script which made the bundle folder empty. It should be there now (ExportToLaravelMigration-SequelAce.zip should show as being 8.1 KB in size).

(And keep in mind: the .sabundle and .spbundle "files" really are just folders anyway that OS X "bundles" into what appears as a file. Sequel Pro registered the .spbundle extension, so Finder will display those as files instead of folders. The beta of Sequel Ace does the same things for .sabundle, but version 2.3.2 doesn't.)