Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#53821 closed update (fixed)

HORTON 2.0.1 : update

Reported by: tczorro (Derrick) Owned by: yangx59@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch maintainer Cc:
Port: horton

Description

Update the old 2.0.0 version to 2.0.1 version

Attachments (1)

change_HORTON_maintainer.diff (2.3 KB) - added by tczorro (Derrick) 7 years ago.

Download all attachments as: .zip

Change History (12)

Changed 7 years ago by tczorro (Derrick)

comment:1 Changed 7 years ago by mf2k (Frank Schima)

Keywords: maintainer removed
Owner: set to yangx59@…
Port: horton added
Status: newassigned
Version: 2.4.1

In the future, please fill in the Port field.

Your email address does not match with the listed maintainer.

comment:2 Changed 7 years ago by mf2k (Frank Schima)

Also, why is your patch modifying the non-existent # $Id$ line? Those were all removed with the switch to GitHub.

Last edited 7 years ago by mf2k (Frank Schima) (previous) (diff)

comment:3 in reply to:  2 Changed 7 years ago by tczorro (Derrick)

Replying to mf2k:

Also, why is your patch modifying the non-existent # $Id$ line? Those were all removed with the switch to GitHub.

Sorry, I just thought it is a new commit so I changed the old information.

comment:4 in reply to:  1 Changed 7 years ago by tczorro (Derrick)

Replying to mf2k:

In the future, please fill in the Port field.

Your email address does not match with the listed maintainer.

I just change the email for my account. The email I listed in the email is my university one.

comment:5 Changed 7 years ago by mf2k (Frank Schima)

Another comment after trying your patch:

  • You are overriding the dependencies and missing python27.
  • Can the dependency on OpenBLAS be changed to allow OpenBLAS-devel?

The following change will fix both. Is that OK with you?

Change:

depends_lib         port:openblas \

to

depends_lib-append  path:lib/libopenblas.dylib:openblas \

comment:6 Changed 7 years ago by mf2k (Frank Schima)

Keywords: maintainer added

comment:7 Changed 7 years ago by mf2k (Frank Schima)

Are you related to the project? While the Mac OS X install page mentions Macports, it does not seem to be aware of the horton port.

comment:8 in reply to:  5 Changed 7 years ago by tczorro (Derrick)

Replying to mf2k:

Another comment after trying your patch:

  • You are overriding the dependencies and missing python27.
  • Can the dependency on OpenBLAS be changed to allow OpenBLAS-devel?

The following change will fix both. Is that OK with you?

Change:

depends_lib         port:openblas \

to

depends_lib-append  path:lib/libopenblas.dylib:openblas \

Thanks for these changes. It is completely fine with me.

comment:9 in reply to:  7 Changed 7 years ago by tczorro (Derrick)

Replying to mf2k:

Are you related to the project? While the Mac OS X install page mentions Macports, it does not seem to be aware of the horton port.

Yes, it is the documentation page of our project. The documentation is more for developers who are willing to make some changes to the source code and build locally for convenience. It will be updated when we plan to release the new version. Then there will be one for normal user through macport and for developers one.

comment:10 Changed 7 years ago by mf2k (Frank Schima)

Resolution: fixed
Status: assignedclosed

In 9699dbcb/macports-ports:

horton:

  • Rename to HORTON and change directory to match
  • Update to version 2.0.1
  • Change depenedency from atlas to openblas

Closes: #53821

comment:11 in reply to:  10 Changed 7 years ago by tczorro (Derrick)

Replying to mf2k:

In 9699dbcb/macports-ports:

horton:

  • Rename to HORTON and change directory to match
  • Update to version 2.0.1
  • Change depenedency from atlas to openblas

Closes: #53821

For the github buildbot error. I think I need to specify the gcc version. For HORTON, it should be 4.9 or above. Could you help revise it.

Note: See TracTickets for help on using tickets.