mingw-cross-env-list
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Mingw-cross-env-list] [PATCH] Openscenegraph updated again.


From: Christian Frisson
Subject: [Mingw-cross-env-list] [PATCH] Openscenegraph updated again.
Date: Mon, 22 Apr 2013 18:16:39 +0200
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:17.0) Gecko/20130328 Thunderbird/17.0.5

Dear Niels and Tony and all,

Last week, I made a CMake patch for OpenSceneGraph that generates openscenegraph-osgPlugin-*.pc files for each plugin, plus an openscenegraph-osgPlugin-all.pc for all, accompanied with osgPlugin/* headers that containt the right USE_OSGPLUGIN(*) call. Note that it can't always be guessed from the plugin dir name, for instance src/osgPlugins/svg/ReaderWriterSVG.cpp states REGISTER_OSGPLUGIN(SVG, ReaderWriterSVG), thus requiring USE_OSGPLUGIN(SVG) not (svg) in your statically-linked app. My patch thus parses all factories to check that. Plus a shell script to compile each/all.

I would like to do a proper git pull request for that, but to do so I'd require that OpenSceneGraph won't get updated so fast. Would it be possible?

Why not just stick to developper releases by the way. I've been using osg for 4 years now, I usually prefered to fork any new/updated plugins instead of recompiling the whole.

Best regards,
Christian Frisson

address@hidden, 22/04/13 18:00:
Send Mingw-cross-env-list mailing list submissions to
        address@hidden

To subscribe or unsubscribe via the World Wide Web, visit
        https://lists.nongnu.org/mailman/listinfo/mingw-cross-env-list
or, via email, send a message with subject or body 'help' to
        address@hidden

You can reach the person managing the list at
        address@hidden

When replying, please edit your Subject line so it is more specific
than "Re: Contents of Mingw-cross-env-list digest..."


Today's Topics:

    1. Re: [PATCH] Openscenegraph updated again. (Tony Theodore)
    2. Re: [PATCH] Update for harfbuzz. (Tony Theodore)
    3. Re: [PATCH] Cherry-picked updates. (Tony Theodore)
    4. Re: Armadillo - cmake global package registry (Christian Frisson)


----------------------------------------------------------------------

Message: 1
Date: Mon, 22 Apr 2013 02:26:24 +1000
From: Tony Theodore <address@hidden>
To: Niels Kristian Bech Jensen <address@hidden>
Cc: "address@hidden"
        <address@hidden>
Subject: Re: [Mingw-cross-env-list] [PATCH] Openscenegraph updated
        again.
Message-ID: <address@hidden>
Content-Type: text/plain; charset="us-ascii"



On 21/04/2013, at 15:05, Niels Kristian Bech Jensen <address@hidden> wrote:

Openscenegraph has been updated once again.

Thanks Niels, there are open issues with osg:

https://github.com/mxe/mxe/issues/11

and it's tracking the development branch. It's not worth updating until those 
issues are resolved.

Cheers,

Tony
-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://lists.nongnu.org/archive/html/mingw-cross-env-list/attachments/20130422/eed2e640/attachment.html>

------------------------------

Message: 2
Date: Mon, 22 Apr 2013 02:30:32 +1000
From: Tony Theodore <address@hidden>
To: Niels Kristian Bech Jensen <address@hidden>
Cc: "address@hidden"
        <address@hidden>
Subject: Re: [Mingw-cross-env-list] [PATCH] Update for harfbuzz.
Message-ID: <address@hidden>
Content-Type: text/plain; charset="us-ascii"



On 20/04/2013, at 18:13, Niels Kristian Bech Jensen <address@hidden> wrote:

The harfbuzz package has been updated to version 0.9.16.

Thanks Niels, this has already been updated:

https://github.com/mxe/mxe/commit/6b8b893327259c606296f23fad50d03a579eb7be

Cheers,

Tony
-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://lists.nongnu.org/archive/html/mingw-cross-env-list/attachments/20130422/db5fa817/attachment.html>

------------------------------

Message: 3
Date: Mon, 22 Apr 2013 19:31:12 +1000
From: Tony Theodore <address@hidden>
To: Niels Kristian Bech Jensen <address@hidden>
Cc: "address@hidden"
        <address@hidden>
Subject: Re: [Mingw-cross-env-list] [PATCH] Cherry-picked updates.
Message-ID: <address@hidden>
Content-Type: text/plain; charset=iso-8859-1


On 15/04/2013, at 8:36 PM, Niels Kristian Bech Jensen <address@hidden> wrote:

Hi Tony.

I have split up the patches as you requested.

Thanks Niels, committed:

https://github.com/mxe/mxe/compare/79760f6...464ae21

It is easier for me to do it by hand using diff and grep than to use git.

Possibly, for some definition of easier - you should try `git add -p`, it may 
surprise you. Using diff/patch increases the overall effort by a factor of 2, 
and processing subsequent changes increases complexity as the reviewer has to 
do a mental `git rebase -i`. This is only a small example, but I'm reminded of 
a great talk by Rich Hickey[1] about how easy/hard are are on different axes 
from simple/complex.

Cheers,

Tony

[1] http://www.infoq.com/presentations/Simple-Made-Easy




------------------------------

Message: 4
Date: Mon, 22 Apr 2013 17:54:04 +0200
From: Christian Frisson <address@hidden>
To: address@hidden
Subject: Re: [Mingw-cross-env-list] Armadillo - cmake global package
        registry
Message-ID: <address@hidden>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed

Dear Niels, Tony, and all,

I replied right where asked:
<https://github.com/mxe/mxe/commit/a147243bafad3ae268ee1586b25f4ef46cfe7514#commitcomment-3060087>

A proposed patch is there.

In short, we can disable the export(PACKAGE... line which mainly creates
CMake config files that I believe useless with CMake through MXE.

Explanation:
<http://cmake.org/Wiki/CMake/Tutorials/Package_Registry>

Generating a pkgconfig file would be more relevant, but that's another
topic.

Best,
Christian Frisson

address@hidden, 20/04/13 10:13:
Send Mingw-cross-env-list mailing list submissions to
        address@hidden

To subscribe or unsubscribe via the World Wide Web, visit
        https://lists.nongnu.org/mailman/listinfo/mingw-cross-env-list
or, via email, send a message with subject or body 'help' to
        address@hidden

You can reach the person managing the list at
        address@hidden

When replying, please edit your Subject line so it is more specific
than "Re: Contents of Mingw-cross-env-list digest..."


Today's Topics:

     1. Re: tryiing to compile Octave (Tony Theodore)
     2. Re: [PATCH] Update dependencies for exiv2       and lensfun.
        (Tony Theodore)
     3. Re: [PATCH] Another set of updates. (Tony Theodore)
     4. Re: [PATCH] New updates for master branch. (Tony Theodore)
     5. Armadillo - cmake global package registry       issue (was: New
        updates for master branch) (Tony Theodore)
     6. Re: [PATCH] New updates for master branch.
        (Niels Kristian Bech Jensen)
     7. [PATCH] Update for harfbuzz. (Niels Kristian Bech Jensen)


----------------------------------------------------------------------

Message: 1
Date: Sat, 20 Apr 2013 14:15:12 +1000
From: Tony Theodore <address@hidden>
To: Alois Schloegl <address@hidden>
Cc: address@hidden
Subject: Re: [Mingw-cross-env-list] tryiing to compile Octave
Message-ID: <address@hidden>
Content-Type: text/plain; charset=iso-8859-1


On 19/04/2013, at 11:19 PM, Alois Schloegl <address@hidden> wrote:

Yes, it is possible. It seems compilation works fine, but there is a
huge number of "undefined references" when trying to link (the log file
has 356493 lines, only the first 7283 lines are related to configuration
and compilation,

There are some minor issues with the configure stage that you want to clean up:

- checking for X... libraries /usr/lib, headers /usr/include
- checking for 
/home/schloegl/src/mxe.github.schloegl/usr/i686-pc-mingw32/bin/pcre-config... no
- configure: WARNING: You used --enable-dl but not --enable-shared.
- configure: WARNING: cURL library not found.  The ftp objects, urlread and 
urlwrite functions will be disabled.
- configure: WARNING: You used the EXPERIMENTAL --enable-openmp option.

Apart from --enable-dl, none of these are likely to be significant, but it 
doesn't hurt to remove the warnings until you get a successful build.

the remaining lines concern "undefined references").
The start and end of that section is shown below. The full log is
available here : http://pub.ist.ac.at/~schloegl/download/octave.log.txt.gz

Sometimes the order of the linked libraries is important. Do you think
that might be the case here, too?

/bin/bash ../libtool --tag=CC   --mode=link i686-pc-mingw32-gcc -g -O2

The vast majority of undefined references [1] seem to be c++ related (e.g. 
'std::', 'operator', '__cxa'). The above should be invoked as 
i686-pc-mingw32-g++.

Cheers,

Tony

[1] grep 'undefined reference' octave.log | gsed -n 's,.*undefined reference to 
\(.*\),\1,p' | sort | uniq




------------------------------

Message: 2
Date: Sat, 20 Apr 2013 14:42:15 +1000
From: Tony Theodore <address@hidden>
To: Niels Kristian Bech Jensen <address@hidden>
Cc: "address@hidden"
        <address@hidden>
Subject: Re: [Mingw-cross-env-list] [PATCH] Update dependencies for
        exiv2   and lensfun.
Message-ID: <address@hidden>
Content-Type: text/plain; charset="iso-8859-1"


On 18/04/2013, at 6:04 PM, Niels Kristian Bech Jensen <address@hidden> wrote:

This patch updates the dependencies for exiv2 (adds gettext) and lensfun (adds 
libgnurx).

Thanks Niels, committed:

https://github.com/mxe/mxe/commit/12c1f2908954c342ad8258a838a07eb4f72b2183
https://github.com/mxe/mxe/commit/5d91f98d1a6b639f541362b922c78961d9415d6a

I changed the exiv2 slightly as libiconv is already required by gettext.

Cheers,

Tony

-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://lists.nongnu.org/archive/html/mingw-cross-env-list/attachments/20130420/f609fac7/attachment.html>

------------------------------

Message: 3
Date: Sat, 20 Apr 2013 14:58:04 +1000
From: Tony Theodore <address@hidden>
To: Niels Kristian Bech Jensen <address@hidden>
Cc: "address@hidden"
        <address@hidden>
Subject: Re: [Mingw-cross-env-list] [PATCH] Another set of updates.
Message-ID: <address@hidden>
Content-Type: text/plain; charset="iso-8859-1"


On 17/04/2013, at 1:16 PM, Niels Kristian Bech Jensen <address@hidden> wrote:

Here is another set of updates. Two of them (openscenegraph and suitesparse) 
are replacements of previous patches.

Thanks Niels, committed:

https://github.com/mxe/mxe/compare/5d91f98...3f28df3

Cheers,

Tony

-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://lists.nongnu.org/archive/html/mingw-cross-env-list/attachments/20130420/cc14c43b/attachment.html>

------------------------------

Message: 4
Date: Sat, 20 Apr 2013 16:29:50 +1000
From: Tony Theodore <address@hidden>
To: Niels Kristian Bech Jensen <address@hidden>
Cc: "address@hidden"
        <address@hidden>
Subject: Re: [Mingw-cross-env-list] [PATCH] New updates for master
        branch.
Message-ID: <address@hidden>
Content-Type: text/plain; charset=iso-8859-1


On 16/04/2013, at 12:43 PM, Niels Kristian Bech Jensen <address@hidden> wrote:

Here are three new updates. Two of them (imagemagick and openscenegraph) are 
meant to replace those I sent before while the third one (gdk-pixbuf) is new.

Thanks, committed without the osg update:
https://github.com/mxe/mxe/compare/3f28df3...46f7c15

Configuring the armadillo package leaves a file in the hidden directory 
.cmake/packages/armadillo which contains the patch to the source directory 
(/home/nkbj/mxe/tmp-armadillo/armadillo-3.800.2). It is the first time I 
encounter this behaviour from cmake and I do not like polution of my home 
directory. The file name is 950318bb345f78e9decf0b3e04f57ba6 (probably 
changing).

It seems to be related to cmake's package registry [1]:

CMakeLists.txt:364
# Export the package for use from the build-tree
# (this registers the build-tree with a global CMake-registry)
if(CMAKE_VERSION VERSION_GREATER "2.7")
   export(PACKAGE armadillo)
endif()

I'm not familiar enough with cmake to remove it, and it would likely affect any 
existing native install, so we certainly want to prevent that. Any ideas?

Cheers,

Tony


[1] http://www.cmake.org/Wiki/CMake/Tutorials/Package_Registry





------------------------------

Message: 5
Date: Sat, 20 Apr 2013 17:00:10 +1000
From: Tony Theodore <address@hidden>
To: "address@hidden"
        <address@hidden>
Subject: [Mingw-cross-env-list] Armadillo - cmake global package
        registry        issue (was: New updates for master branch)
Message-ID: <address@hidden>
Content-Type: text/plain; charset=iso-8859-1


On 20/04/2013, at 4:29 PM, Tony Theodore <address@hidden> wrote:

On 16/04/2013, at 12:43 PM, Niels Kristian Bech Jensen <address@hidden> wrote:

Configuring the armadillo package leaves a file in the hidden directory 
.cmake/packages/armadillo which contains the patch to the source directory 
(/home/nkbj/mxe/tmp-armadillo/armadillo-3.800.2). It is the first time I 
encounter this behaviour from cmake and I do not like polution of my home 
directory. The file name is 950318bb345f78e9decf0b3e04f57ba6 (probably 
changing).

It seems to be related to cmake's package registry [1]:

CMakeLists.txt:364
# Export the package for use from the build-tree
# (this registers the build-tree with a global CMake-registry)
if(CMAKE_VERSION VERSION_GREATER "2.7")
export(PACKAGE armadillo)
endif()

I'm not familiar enough with cmake to remove it, and it would likely affect any 
existing native install, so we certainly want to prevent that. Any ideas?

Cheers,

Tony


[1] http://www.cmake.org/Wiki/CMake/Tutorials/Package_Registry

I've disabled the build for the time being and asked the author of the original 
submission to take a look:

https://github.com/mxe/mxe/commit/a147243bafad3ae268ee1586b25f4ef46cfe7514#commitcomment-3051877

Cheers,

Tony




------------------------------

Message: 6
Date: Sat, 20 Apr 2013 10:07:36 +0200
From: Niels Kristian Bech Jensen <address@hidden>
To: "address@hidden"
        <address@hidden>
Subject: Re: [Mingw-cross-env-list] [PATCH] New updates for master
        branch.
Message-ID: <address@hidden>
Content-Type: text/plain; charset="iso-8859-1"

It seems to be related to cmake's package registry [1]:

CMakeLists.txt:364
# Export the package for use from the build-tree
# (this registers the build-tree with a global CMake-registry)
if(CMAKE_VERSION VERSION_GREATER "2.7")
   export(PACKAGE armadillo)
endif()

I'm not familiar enough with cmake to remove it, and it would likely affect any 
existing native install, so we certainly want to prevent that. Any ideas?

I think it should be removed since it points to the build tree which is deleted 
by MXE when armadillo is installed. That creates a dangling pointer in the 
cmake registry. I have attached a patch to do that. Best regards,Niels Kristian 
Bech Jensen
                                        
-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://lists.nongnu.org/archive/html/mingw-cross-env-list/attachments/20130420/5276262a/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: armadillo-2-do-not-export-package.patch
Type: application/octet-stream
Size: 746 bytes
Desc: not available
URL: 
<http://lists.nongnu.org/archive/html/mingw-cross-env-list/attachments/20130420/5276262a/attachment.obj>




------------------------------

_______________________________________________
Mingw-cross-env-list mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/mingw-cross-env-list


End of Mingw-cross-env-list Digest, Vol 44, Issue 22
****************************************************





reply via email to

[Prev in Thread] Current Thread [Next in Thread]