Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#40795 closed update (fixed)

gwyddion @2.33 with mac integration

Reported by: nano103 Owned by: rowue@…
Priority: Normal Milestone:
Component: ports Version: 2.2.0
Keywords: Cc:
Port: gwyddion

Description

  • version increased to 2.33
  • xz archive instead of tgz
  • gtk-osx-application dependency in quartz variant
  • patches to sources

Attachments (5)

build_fail.log (23.5 KB) - added by rowue@… 11 years ago.
Log from failed build
Portfile.diff (699 bytes) - added by nano103 11 years ago.
New revision patch against 2.33 Portfile
patch-app-mac_integration.c.diff (6.1 KB) - added by nano103 11 years ago.
patch-configure.diff (9.4 KB) - added by nano103 11 years ago.
patch-app-Makefile.in.diff (4.3 KB) - added by nano103 11 years ago.

Download all attachments as: .zip

Change History (21)

comment:1 Changed 11 years ago by rowue@…

Owner: changed from macports-tickets@… to rowue@…
Status: newassigned

comment:2 Changed 11 years ago by rowue@…

Hi Andrey,

thanks a lot for the patch to the portfile and the work on the patches. However I have some questions regarding your patch on the Portfile:

  1. You apply the patches in any case - as far as I understand, they will only be usefull if someone uses quartz ... - however since they will be in 2.34 we should check them now ;)
  2. You use "depends_lib" in the quartz section, overriding prior dependencies (f.e. fftw3) is this wanted (I don't think so)

KInd regards,

Rolf

comment:3 in reply to:  2 ; Changed 11 years ago by nano103

Hello, Rolf

I make a test portfile to build gwyddion from trunk and it works fine, so all patches are accepted to gwyddion and only dependency gtk-osx-application should be inserted into 2.34 portfile after next gwyddion release.

Integration patch completely removes dependence of Carbon both in X11 and Quartz variants so it is basically quartz but not quartz only.

In quartz section I meant "add dependency", perhaps I was wrong in portfile sintax ?

Moreover I suppose gtk-osx-application going to be optional after some fixes in trunk, so dependency insertion into Portfile will be unnecessary i.e if gtk-osx-application already installed it will be used

And I have one question: why portfile warns about broken Quartz GL in 10.8 ? After fixing integration it builds and runs fine and I did not find any problems with GL.

comment:4 in reply to:  3 Changed 11 years ago by rowue@…

Replying to nano103@…:

Hello, Rolf

Hi Andrey,

I make a test portfile to build gwyddion from trunk and it works fine, so all patches are accepted to gwyddion and only dependency gtk-osx-application should be inserted into 2.34 portfile after next gwyddion release.

Yes, that is what I also got from gwyddion-dev

Integration patch completely removes dependence of Carbon both in X11 and Quartz variants so it is basically quartz but not quartz only.

In quartz section I meant "add dependency", perhaps I was wrong in portfile sintax ?

It seems so ;)

Moreover I suppose gtk-osx-application going to be optional after some fixes in trunk, so dependency insertion into Portfile will be unnecessary i.e if gtk-osx-application already installed it will be used

If I get this right, gtk-osx-application is the dependency to get gwyddion running with quartz and 10.8 - so we should keep this as a dependency for the quartz variant (so the user can simply choose +quartz and get the native OS X backend)

And I have one question: why portfile warns about broken Quartz GL in 10.8 ? After fixing integration it builds and runs fine and I did not find any problems with GL.

Simply because I don't manage to get gwyddion compiling with +quartz. It fails on a dependency: py27-cairo. (So in principle I should remove the quartz section and/or file an error)

comment:5 Changed 11 years ago by rowue@…

for py27-cairo see also: #38606

comment:6 in reply to:  5 Changed 11 years ago by nano103

Replying to rowue@…:

for py27-cairo see also: #38606

I see - fresh macports installation will not help to reproduce this problem because it solves the problem. May be switch from X11 to Quartz will help. My installation was is initially with +quartz and +no_x11.

comment:7 Changed 11 years ago by rowue@…

I did "port clean all; port uninstall installed" and afterwards tried to install gwyddion with "+quartz +no_x11" (ususal test) - it fails at py27-cairo. From this I would assign the "upgrade" error mentioned in #38606 to an upgrade of macports. I can try with a fresh installation (so we can test and recommend) tomorrow evening ....

comment:8 Changed 11 years ago by rowue@…

After removing /opt/local installing macports 2.2.0 from dmg py27-cairo works as expected.

Now building of gwyddion (+quartz +no_x11) fails - see log. If I check the output from configure it seems, that your patches don't belong to 2.33:

--->  Applying patches to gwyddion
--->  Applying patch-configure.diff
patching file configure
Hunk #1 succeeded at 19348 (offset 13 lines).
--->  Applying patch-app-mac_integration.c.diff
patching file app/mac_integration.c
--->  Applying patch-app-Makefile.in.diff
patching file app/Makefile.in
patch unexpectedly ends in middle of line
Hunk #1 succeeded at 525 with fuzz 1.
--->  Configuring gwyddion

Can you send the patches against 2.33 (or should I look at the ML?)

Changed 11 years ago by rowue@…

Attachment: build_fail.log added

Log from failed build

comment:9 in reply to:  8 Changed 11 years ago by nano103

Is your plan to upgrade 2.33 Portfile? If so I can recreate patch for 2.33 with latest improvements from trunk.

If you are building from trunk this patches are accepted and should be removed from portfile And gtk-osx-application dependency also should be removed - it is optional now.

Anyway patch-app-Makefile.in.diff is only for build warnings removing

comment:10 Changed 11 years ago by rowue@…

My plan is to provide a working (compiling and running) version of gwyddion 2.33 with quartz. So it would be very nice if you can hand me the patches needed to integrate the gwyddion ui into quartz on a 10.8 system. And yes, if necessary I would also update the portfile (which I should do because if gl works (as you stated) we don't need the ui-msg any more) (This sounds more harsh than it should)

I think for the last patches there was only a slight difference which keeps them away from working, so maybe a new diff will help ...

Kind regards,

Rolf

PS: For the py27-cario stuff it seems, that "-x11" is the flag someone needs to get things working with "no_x11" and "quartz"

comment:11 Changed 11 years ago by nano103

OK, I'll provide new patch tomorrow And I think the problem in Makefile.in path is a lack of EOL in last line

Changed 11 years ago by nano103

Attachment: Portfile.diff added

New revision patch against 2.33 Portfile

Changed 11 years ago by nano103

Changed 11 years ago by nano103

Attachment: patch-configure.diff added

Changed 11 years ago by nano103

Attachment: patch-app-Makefile.in.diff added

comment:12 Changed 11 years ago by nano103

Patches are updated

comment:13 Changed 11 years ago by rowue@…

Resolution: fixed
Status: assignedclosed

committed patches in r112505

comment:14 Changed 11 years ago by rowue@…

as stated above, committed - ticket closed

works like a charm -

Andrey: Thanks a lot

comment:15 in reply to:  13 Changed 11 years ago by cooljeanius (Eric Gallager)

Replying to rowue@…:

committed patches in r112505

I think you still have to make the portfile actually use the patches with patchfiles-append still...

comment:16 Changed 11 years ago by rowue@…

They are active if you choose the quartz variant.

Note: See TracTickets for help on using tickets.