New Ticket     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #31588 (closed update: fixed)

Opened 7 months ago

Last modified 3 months ago

grisbi @0.6.0 update to version 0.8.8

Reported by: gerald@… Owned by: ryandesign@…
Priority: Normal Milestone:
Component: ports Version: 2.0.3
Keywords: haspatch maintainer Cc: devans@…
Port: grisbi

Description

Update Portfile to version 0.8.8 of Grisbi.

Attachments

Portfile-grisbi.diff Download (1.5 KB) - added by gerald@… 7 months ago.
patch-grisbi-Portfile.diff Download (2.0 KB) - added by devans@… 6 months ago.
Revised patch for Portfile
grisbi-ryandesign.diff Download (2.2 KB) - added by ryandesign@… 6 months ago.
proposed patch

Change History

Changed 7 months ago by gerald@…

Changed 7 months ago by ryandesign@…

  • owner changed from macports-tickets@… to ryandesign@…
  • status changed from new to assigned
  • port set to grisbi
  • keywords haspatch maintainer added

Thanks. There are several changes I'd suggest. I'll attach a new patch and explain and you can see what you think.

Changed 6 months ago by devans@…

Since this has been sitting for a while, here is an updated patch for the Portfile for review and comment by both of you (see attached)

  • remove configure.args that are either the default or no longer recognized
  • add --disable-silent-rules to make debugging easier
  • add build dependencies on pkgconfig and intltool which are required
  • convert lib-style lib dependencies to port dependencies to ensure only using MacPorts versions of these libraries
  • remove universal_variant yes -- this is the default
  • remove supported archs statement since there doesn't seem to be any restriction on arch
  • fix livecheck
  • add license

Ryan may have some other suggestions

Changed 6 months ago by devans@…

Revised patch for Portfile

Changed 6 months ago by devans@…

  • cc devans@… added

Cc Me!

Changed 6 months ago by ryandesign@…

Sorry, I forgot about this. Additional comments:

  • master_sites sourceforge should be rewritten to avoid redirects
  • the no_ofx variant should be deprecated and turned into an on-by-default ofx variant; the no_ofx variant can be deleted in a year or so once everyone has upgraded to this new version
  • the livecheck should be able to discover new stable branches of the software
  • we prefer rmd160 & sha256 checksum types

I've attached a new update, which is almost ready, except that grisbi has a bug where it will use ofx, even if you ask it not to; I filed  upstream bug 1432 about this.

Ideally I'd also like to make a second commit later to fix the port's whitespace and formatting to conform to the modeline and our usual formatting practices.

Changed 6 months ago by ryandesign@…

proposed patch

Changed 3 months ago by raimue@…

  • status changed from assigned to closed
  • resolution set to fixed

Committed proposed patch in r90310.

Note: See TracTickets for help on using tickets.