Bug 191888 - lang/python27: Remove PTH option
Summary: lang/python27: Remove PTH option
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: Marcus von Appen
URL: https://phabric.freebsd.org/D488
Keywords:
Depends on:
Blocks: 191744
  Show dependency treegraph
 
Reported: 2014-07-15 21:20 UTC by Jung-uk Kim
Modified: 2014-08-03 07:53 UTC (History)
2 users (show)

See Also:


Attachments
Patch to remove the PTH option (1.50 KB, patch)
2014-07-15 21:20 UTC, Jung-uk Kim
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jung-uk Kim freebsd_committer 2014-07-15 21:20:57 UTC
Created attachment 144704 [details]
Patch to remove the PTH option

PTH option (GNU Pth support) is known to cause trouble and it should be removed[1].  devel/pth installs pthread.h by default, which implements POSIX pthread.h via GNU Pth.  This file defines several mandatory types.  Unfortunately, some types must be defined via sys/types.h per POSIX[2].

http://pubs.opengroup.org/onlinepubs/009695399/basedefs/sys/types.h.html

Therefore, if a file includes sys/types.h and pthread.h at the same time, it obviously redefines these types.  Especially, it is a fatal error for clang.

[1] Actually, the option was completely removed in recent Python releases.
[2] We actually defined them in sys/_pthreadtypes.h and let sys/types.h include it.  See Bug 78907.
Comment 1 Marcus von Appen freebsd_committer freebsd_triage 2014-07-27 16:20:33 UTC
I'll handle this with the lang/python27 update at https://phabric.freebsd.org/D488
Comment 2 Marcus von Appen freebsd_committer freebsd_triage 2014-08-03 07:53:04 UTC
The PTH option has been removed in r363790.