Re: Patch for pgAdmin4 package on Mac OS X - Mailing list pgadmin-hackers

From Sandeep Thakkar
Subject Re: Patch for pgAdmin4 package on Mac OS X
Date
Msg-id CANFyU96f4a9Zn2czw3Ch_k0w1cyiwiY4KDdyScQgWnW9vQsM-A@mail.gmail.com
Whole thread Raw
In response to Re: Patch for pgAdmin4 package on Mac OS X  (Dave Page <dpage@pgadmin.org>)
Responses Re: Patch for pgAdmin4 package on Mac OS X  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Thanks Dave. 

Please see inline.

On Fri, May 6, 2016 at 9:03 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

Thanks. I've applied the path fix patch. The second one needs a little
work - I've attached an updated version to work from:

- I've updated the README, and some of the default values.

OK. The default values of PGDIR and QTDIR set by you is different than mine. I installed them through macports.
 
- I've removed the file type registration for .sql files.

OK. 
 
 

- Should we note that the user may need to run in a virtualenv?

No, we bundle private environment, right? 
 

- Please move build-mac.sh to pkg/build.sh, and create a target in /Makefile to
  execute it, e.g. "make appbundle"

Sure. You mean move to pkg/mac/build.sh, right? 

- Extend the Makefile to add a "clean-appbundle" target, which should also be
  called by the "clean" target.

OK. 
- At present, it is bundling my pre-existing configuration database. It *must*
  create a new one and bundle that, without touching the existing one (I guess
  that may require a new command line option for setup.py).

No, it's not bundling the pgadmin4.db at all, I skipped it :). Paresh is working on this. 

- The online help is broken (are you building it)? I suggest adding a top-level
  Makefile target to do so.

Sorry, which online help?
 
- Working directories should be added to /.gitignore. Please ensure they don't
  clash with those used by pip (and ideally are in one place, e.g. mac-build/).

OK.
 
- I saw various errors in the build output, though the resulting DMG seemed to
  work fine;

Yeah, because I had libpq.dylib present in $PGDIR/ and may be yours present in $PGDIR/lib/. I will be using the default values suggested by you and fix this. Thanks!

...
cp: /usr/local/pgsql/libpq.5.dylib: No such file or directory
libpq.5.dylib not found in /usr/local/pgsql
Completing app: /Users/dpage/git/pgadmin4/pgAdmin4.app
...

...
App: pgAdmin4.app: Post-processing: .//Contents/MacOS/pgAdmin4
App: pgAdmin4.app: Adding symlink: QtWebKitWidgets (because of:
.//Contents/MacOS/pgAdmin4)
cp: @rpath/QtWebKitWidgets.framework/Versions/5/../../../QtWebKitWidgets.framework:
No such file or directory
chmod: Contents/Frameworks/QtWebKitWidgets.framework/Versions/5/QtWebKitWidgets:
No such file or directory
Rewriting ID in
Contents/Frameworks/QtWebKitWidgets.framework/Versions/5/QtWebKitWidgets
to QtWebKitWidgets
error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/install_name_tool:
can't open file:
Contents/Frameworks/QtWebKitWidgets.framework/Versions/5/QtWebKitWidgets
(No such file or directory)
Cleaning up
...


On Mon, Apr 18, 2016 at 1:25 PM, Sandeep Thakkar
<sandeep.thakkar@enterprisedb.com> wrote:
> Hi Team, Dave,
>
> Attached herewith are two patches.
>
> pgadmin4-mac-bundle.patch - This includes scripts to build Mac app bundle
> and DMG for pgAdmin4. This is the tree of the generated app bundle:
>
> ----
> $ find pgAdmin4.app/ -maxdepth 3 -type d
> pgAdmin4.app/
> pgAdmin4.app//ContentspgAdmin4.app//Contents/Frameworks
> pgAdmin4.app//Contents/Frameworks/Python.framework
> pgAdmin4.app//Contents/Frameworks/QtCore.framework
> pgAdmin4.app//Contents/Frameworks/QtDBus.framework
> pgAdmin4.app//Contents/Frameworks/QtGui.framework
> pgAdmin4.app//Contents/Frameworks/QtMultimedia.framework
> pgAdmin4.app//Contents/Frameworks/QtMultimediaWidgets.framework
> pgAdmin4.app//Contents/Frameworks/QtNetwork.framework
> pgAdmin4.app//Contents/Frameworks/QtOpenGL.framework
> pgAdmin4.app//Contents/Frameworks/QtPositioning.framework
> pgAdmin4.app//Contents/Frameworks/QtPrintSupport.framework
> pgAdmin4.app//Contents/Frameworks/QtQml.framework
> pgAdmin4.app//Contents/Frameworks/QtQuick.framework
> pgAdmin4.app//Contents/Frameworks/QtSensors.framework
> pgAdmin4.app//Contents/Frameworks/QtSql.framework
> pgAdmin4.app//Contents/Frameworks/QtWebChannel.framework
> pgAdmin4.app//Contents/Frameworks/QtWebKit.framework
> pgAdmin4.app//Contents/Frameworks/QtWebKitWidgets.framework
> pgAdmin4.app//Contents/Frameworks/QtWidgets.framework
> pgAdmin4.app//Contents/MacOS
> pgAdmin4.app//Contents/PlugIns
> pgAdmin4.app//Contents/PlugIns/platforms
> pgAdmin4.app//Contents/Resources
> pgAdmin4.app//Contents/Resources/venv
> pgAdmin4.app//Contents/Resources/web
> ---
>
> pgadmin4-fixpath-mac.patch - This is for runtime/Server.cpp to fix the path
> where the runtime looks for web application path in app bundle.
>
>
> Kindly review and suggest the changes required.  Thanks.
>
> --
> Sandeep Thakkar
>



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
Sandeep Thakkar

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: PATCH: FTS configuration node
Next
From: Dave Page
Date:
Subject: Re: Patch for pgAdmin4 package on Mac OS X