Bug 249175 - mail/getmail6: fix build errors on package-as-user
Summary: mail/getmail6: fix build errors on package-as-user
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Joseph Mingrone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-09-07 18:25 UTC by Michael Büker
Modified: 2020-09-28 08:53 UTC (History)
2 users (show)

See Also:
freebsd: maintainer-feedback+


Attachments
getmail6-buildfix (474 bytes, patch)
2020-09-07 18:25 UTC, Michael Büker
freebsd: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Büker 2020-09-07 18:25:10 UTC
Created attachment 217803 [details]
getmail6-buildfix

This port fails to build as user -- see http://package19.nyi.freebsd.org/data/121amd64-default-build-as-user/547766/logs/py37-getmail6-6.7.log

This is because in python3, __pycache__ files are always created with root as owner. Therefore, the @${CHMOD} command in post-install: fails when run as user.

When this port was forked from mail/getmail (which was purely python2), this error did not show up in testing and review.

Upon closer inspection, the @${CHMOD} operation is superfluous. File permissions in the final package are fine without it.

Therefore, this patch removes the post-install: @${CHMOD} operation.
Comment 1 commit-hook freebsd_committer freebsd_triage 2020-09-28 08:52:55 UTC
A commit references this bug:

Author: jrm
Date: Mon Sep 28 08:52:38 UTC 2020
New revision: 550410
URL: https://svnweb.freebsd.org/changeset/ports/550410

Log:
  mail/getmail6: fix build errors on package-as-user

  This build fails as user, because in python3 __pycache__ files are always
  created with root as owner.  Therefore, the @${CHMOD} command in
  post-install: fails when run as user.  Upon closer inspection, the
  @${CHMOD} operation is superfluous. File permissions in the final package
  are fine without it.

  PR:		249175
  Submitted by:	m.bueker@berlin.de (maintainer)

Changes:
  head/mail/getmail6/Makefile