Bug 247299 - mail/gmime30: Add VALA option
Summary: mail/gmime30: Add VALA option
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-gnome (Nobody)
URL:
Keywords: needs-qa
Depends on:
Blocks:
 
Reported: 2020-06-16 09:07 UTC by Bertram Scharpf
Modified: 2020-07-11 08:20 UTC (History)
2 users (show)

See Also:
koobs: maintainer-feedback? (gnome)


Attachments
Patch with vala option (1.25 KB, patch)
2020-06-16 09:07 UTC, Bertram Scharpf
no flags Details | Diff
Patch with options helper syntax (1.18 KB, patch)
2020-06-18 18:58 UTC, Bertram Scharpf
no flags Details | Diff
Gmime30 patch with vala option and vala default (1.20 KB, patch)
2020-06-26 19:11 UTC, Bertram Scharpf
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bertram Scharpf 2020-06-16 09:07:54 UTC
Created attachment 215602 [details]
Patch with vala option

On a non-graphic system I have installed +neomutt+. This pulls +gmime30+ what now tries to install +vala+. I do not want this.

I propose an option to deactivate the Vala lib bindings.

I tested my patch on the non-graphical system and on a graphical one as well.
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2020-06-16 09:15:20 UTC
Thank you for the report and patch Bertram.

The .if ${PORT_OPTIONS:MVALA} block should be converted to the options helper syntax

VALA_BUILD_DEPENDS=
VALA_CONFIGURE_OFF

But I would also try just:

VALA_BUILD_DEPENDS=foo
VALA_CONFIGURE_ENABLE=vala (configure needs to support --disable-vala, which some dont)
Comment 2 Bertram Scharpf 2020-06-18 18:58:43 UTC
Created attachment 215745 [details]
Patch with options helper syntax

Does this new patch please you more?

I just copied it from +mail/gmime26+ and wonder why this was removed in version 30.
Comment 3 Ting-Wei Lan 2020-06-23 00:37:43 UTC
(In reply to Bertram Scharpf from comment #2)
Can you keep vala enabled by default, so the existing behavior isn't changed?
Comment 4 Bertram Scharpf 2020-06-26 19:11:41 UTC
Created attachment 215967 [details]
Gmime30 patch with vala option and vala default

Next try...
Comment 5 Bertram Scharpf 2020-07-06 02:36:05 UTC
Reminder.
Comment 6 Bertram Scharpf 2020-07-11 08:20:00 UTC
Reminder.