1Password mysql shell plugin breaks additional CLI arguments #406

Activity

1. nr-dbuckwalter commented on Nov 30, 2023

   

   nr-dbuckwalter

   on Nov 30, 2023

   More actions

   @minaguib I was facing the same issue and have sumbitted a fairly simple PR to address it (works for me!). Can you give it a try as well?
2. minaguib commented on Nov 30, 2023

   

   minaguib

   on Nov 30, 2023

   Last edited by minaguib

   Author

   More actions

   @nr-dbuckwalter Looks good on my end - thanks :)

   For reference, for anyone else who wants to try:

   # Clone Damon's branch:
   git clone git@github.com:nr-dbuckwalter/shell-plugins.git -b fix-mysql-arg-order
   
   # Build the mysql plugin locally
   cd shell-plugins
   make mysql/build
   
   # Validate it got installed here:
   $ ls -la ~/.config/op/plugins/local/
   -rwxr-xr-x  1 mina  staff  20390690 30 Nov 16:19 mysql
   
   # Test it - you'll get an ugly warning due to the above setup till properly released
   $ mysql --version
   #############################################################################
   # WARNING: 'mysql' is not from the official registry.                       #
   # Only proceed if you are the developer of 'mysql'.                         #
   # Otherwise, delete the file at /Users/mina/.config/op/plugins/local/mysql. #
   #############################################################################
   mysql  Ver 8.2.0 for macos14.0 on arm64 (Homebrew)
   
3. gccervonebport commented on Jun 30, 2024

   

   gccervonebport

   on Jun 30, 2024

   More actions

   @nr-dbuckwalter
   Thanks so much for your work on this - now mysql is fully useable. And, I was able to get a plugin working for mysqldump too - which I really needed!

   -= G =-