Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#16852 closed submission (fixed)

py25-shogun-0.6.6 New port of A Large Scale Machine Learning Toolbox

Reported by: stromnov (Andrew Stromnov) Owned by: nerdling (Jeremy Lavergne)
Priority: Normal Milestone:
Component: ports Version: 1.7.0
Keywords: Cc:
Port: py25-shogun

Description

Port of "Shogun - A Large Scale Machine Learning Toolbox" (http://www.shogun-toolbox.org/)

Attachments (9)

Portfile (875 bytes) - added by stromnov (Andrew Stromnov) 12 years ago.
python/py25-shogun/Portfile
Portfile.2 (2.8 KB) - added by krunk7@… 12 years ago.
Portfile-modified with patch
patch-src.diff (1.8 KB) - added by krunk7@… 12 years ago.
Patch for configure
Portfile.3 (3.2 KB) - added by nerdling (Jeremy Lavergne) 12 years ago.
cleaned up lint warnings, added portgroup, should we disable python variant?
py25-shogun.log (8.1 KB) - added by nerdling (Jeremy Lavergne) 12 years ago.
debug info from building
Portfile.4 (2.8 KB) - added by krunk7@… 12 years ago.
My Working portfile
patch-src (1.8 KB) - added by krunk7@… 12 years ago.
For working portfile
patch-src-lib-lapack.cpp (363 bytes) - added by krunk7@… 12 years ago.
another for working portfile
Portfile.5 (2.8 KB) - added by nerdling (Jeremy Lavergne) 12 years ago.
patch phase changes (added patchfiles, removed override)

Download all attachments as: .zip

Change History (23)

Changed 12 years ago by stromnov (Andrew Stromnov)

Attachment: Portfile added

python/py25-shogun/Portfile

Changed 12 years ago by krunk7@…

Attachment: Portfile.2 added

Portfile-modified with patch

Changed 12 years ago by krunk7@…

Attachment: patch-src.diff added

Patch for configure

comment:1 Changed 12 years ago by krunk7@…

Any status on this ports acceptance?

comment:2 Changed 12 years ago by nerdling (Jeremy Lavergne)

Owner: changed from macports-tickets@… to snc@…
Status: newassigned
Version: 1.6.01.7.0

I'm going through the portfile presently.

--->  Verifying Portfile for py25-shogun
Warning: Line 1 is missing RCS tag ($Id$)
Warning: Line 3 should be a newline (after PortSystem)
--->  0 errors and 2 warnings found.

Changed 12 years ago by nerdling (Jeremy Lavergne)

Attachment: Portfile.3 added

cleaned up lint warnings, added portgroup, should we disable python variant?

comment:3 Changed 12 years ago by nerdling (Jeremy Lavergne)

Does the new portfile work for you?

comment:4 Changed 12 years ago by nerdling (Jeremy Lavergne)

This failed to build for me.

/opt/local/Library/Frameworks/Python.framework/Versions/2.5/Resources/Python.app/Contents/MacOS/Python: can't open file 'setup.py': [Errno 2] No such file or directory
Warning: the following items did not execute (for py25-shogun): org.macports.activate org.macports.build org.macports.destroot org.macports.archive org.macports.install

Changed 12 years ago by nerdling (Jeremy Lavergne)

Attachment: py25-shogun.log added

debug info from building

comment:5 Changed 12 years ago by stromnov (Andrew Stromnov)

"PortGroup python25" redefines build sequence. Add "use_configure yes" to Portfile.

comment:6 in reply to:  5 Changed 12 years ago by nerdling (Jeremy Lavergne)

Replying to stromnov@…:

Add "use_configure yes" to Portfile.

Isn't the default already yes?

comment:7 Changed 12 years ago by krunk7@…

Fixed by adding the py25-setuptools as a dependency. No portgroup used.

New files attached.

Changed 12 years ago by krunk7@…

Attachment: Portfile.4 added

My Working portfile

Changed 12 years ago by krunk7@…

Attachment: patch-src added

For working portfile

Changed 12 years ago by krunk7@…

Attachment: patch-src-lib-lapack.cpp added

another for working portfile

comment:8 Changed 12 years ago by nerdling (Jeremy Lavergne)

A couple comments on Portfile.4:

Patchfiles should follow the source patch naming policy "patch-*.diff" The patch phase does patch -p0 < based on the patchfiles directive.

Changed 12 years ago by nerdling (Jeremy Lavergne)

Attachment: Portfile.5 added

patch phase changes (added patchfiles, removed override)

comment:9 Changed 12 years ago by krunk7@…

Anything else needed to push this port live?

comment:10 Changed 12 years ago by nerdling (Jeremy Lavergne)

Regarding the ui_msg: Make sure that /opt/local/lib is in your DYLD_LIBRARY_PATH or the sg module will not load properly!

You may consider having the binaries moved to ${prefix}/libexec and create wrapper scripts that handle the DYLD path and call the binaries in ${prefix}/bin.

I'll go ahead and commit what we have. If you'd like this change made, let me know or create another ticket.

comment:11 Changed 12 years ago by nerdling (Jeremy Lavergne)

Resolution: fixed
Status: assignedclosed

Committed, r47003.

comment:12 Changed 12 years ago by nerdling (Jeremy Lavergne)

Actually, we do need to create a new ticket. There are some /opt/local paths hard coded into the patches.

These need to be replaced with the reinplace command.

comment:13 Changed 12 years ago by jmroot (Joshua Root)

Type: enhancementsubmission

comment:14 Changed 12 years ago by (none)

Milestone: Port Submissions

Milestone Port Submissions deleted

Note: See TracTickets for help on using tickets.