Opened 13 years ago

Closed 13 years ago

#28524 closed update (fixed)

caml-graph @1.3 upgrade to version 1.6

Reported by: fh@… Owned by: avsm@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: haspatch Cc: ryandesign (Ryan Carsten Schmidt)
Port: caml-graph

Description

Current version (1.3) fails during the destroot phase (see ticket #26091). I have attached files that upgrade to version 1.6. It fixes the bug at the same time.

Attachments (4)

Portfile-caml-graph.diff (2.5 KB) - added by fh@… 13 years ago.
Portfile patch
patch-Makefile.in (459 bytes) - added by fh@… 13 years ago.
Portfile-caml-graph.2.diff (997 bytes) - added by fh@… 13 years ago.
patch-Makefile.in.diff (739 bytes) - added by fh@… 13 years ago.

Download all attachments as: .zip

Change History (10)

Changed 13 years ago by fh@…

Attachment: Portfile-caml-graph.diff added

Portfile patch

Changed 13 years ago by fh@…

Attachment: patch-Makefile.in added

comment:1 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added

Your diff is difficult to evaluate and accept. You claim your patch updates the port to version 1.6, but you also make all sorts of unrelated changes that I don't understand:

  • you remove the detailed long_description
  • you change "maintainers nomaintainer" (which is correct) to just "maintainers" (which is invalid)
  • you remove the slash at the end of the homepage and master_sites, making them uncanonical (not a major problem, but certainly less correct than before)
  • you move various lines around, like the distname and livecheck lines
  • you change the name of the patchfile from "patch-Makefile.in.diff" (our patchfile preferred format) to "patch-Makefile.in"
  • you disable the universal variant (this change is probably correct, but not particularly related to updating the port to 1.6, is it?)

Perhaps you could attach a patch that only updates the port to 1.6 (including updating the patch contents along with it, if applicable, and other portfile changes that directly relate to updating to 1.6). If you have other changes you'd like done to the port, such as reformatting, we could handle those separately later.

comment:2 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from macports-tickets@… to avsm@…

comment:3 in reply to:  1 ; Changed 13 years ago by fh@…

Replying to ryandesign@…:

Thanks for all you comments. I have attached new patches that hopefully match the requirements. Some important points:

  • I have updated to version 1.7 that went out yesterday. The new version fixes a bug in Makefile.in that makes my previous patch to Makefile.in irrelevant.
  • My previous patch to Portfile was totally wrong as the files were not destrooted but installed directly in the macports tree. I went back to the building process of version 1.3. Still my new patch to Makefile.in differs from the current one (@1.3) as ocamlfind was not invoked correctly in case LABLGNOMECANVAS was selected.
  • you disable the universal variant (this change is probably correct, but not particularly related to updating the port to 1.6, is it?)

I believe it should be inherited from the caml port, but I agree it has nothing to do in that particular update.

Changed 13 years ago by fh@…

Attachment: Portfile-caml-graph.2.diff added

Changed 13 years ago by fh@…

Attachment: patch-Makefile.in.diff added

comment:4 Changed 13 years ago by fh@…

Cc: fh@… added

Cc Me!

comment:5 in reply to:  3 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: fh@… removed

Replying to fh@…:

  • you disable the universal variant (this change is probably correct, but not particularly related to updating the port to 1.6, is it?)

I believe it should be inherited from the caml port, but I agree it has nothing to do in that particular update.

Ah yes, ocaml doesn't have a universal variant, so caml-graph shouldn't either. I made this change in r78984.

comment:6 in reply to:  3 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Replying to fh@…:

  • I have updated to version 1.7 that went out yesterday.

Thanks, I've updated the port in r78985.

Note: See TracTickets for help on using tickets.