Bug 170290 - devel/pylint broken after latest update due to permissions issue
Summary: devel/pylint broken after latest update due to permissions issue
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Marcus von Appen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-31 03:50 UTC by Enji Cooper
Modified: 2013-02-03 22:30 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Enji Cooper freebsd_committer freebsd_triage 2012-07-31 03:50:10 UTC
pylint won't run after a recent astng2 update, because the file permissions for the py2stdlib module are too exclusive:

$ pylint
Traceback (most recent call last):
  File "/usr/local/bin/pylint", line 3, in <module>
    from pylint import lint
  File "/usr/local/lib/python2.7/site-packages/pylint/lint.py", line 31, in <module>
    from pylint.checkers import utils
  File "/usr/local/lib/python2.7/site-packages/pylint/checkers/__init__.py", line 44, in <module>
    from logilab.astng.utils import ASTWalker
  File "/usr/local/lib/python2.7/site-packages/logilab/astng/__init__.py", line 85, in <module>
    __import__(module[:-3])
ImportError: No module named py2stdlib
$ ls -l /usr/local/lib/python2.7/site-packages
-rw-------  1 root  wheel  2828 Jul 18 02:38 /usr/local/lib/python2.7/site-packages/logilab/astng/brain/py2stdlib.py

After I chmod a+r the file, then pylint functions again:

$ pylint
Usage:  pylint [options] module_or_package

  Check that a module satisfy a coding standard (and more !).

    pylint --help

  Display this help message and exit.

    pylint --help-msg <msg-id>[,<msg-id>]

  Display help messages about given message identifiers and exit.


Options:
  --version             show program's version number and exit
  -h, --help            show this help message and exit
  --long-help           more verbose help.

  Master:
    --rcfile=<file>     Specify a configuration file.
    -E, --errors-only   In error mode, checkers without error messages are
                        disabled and for others, only the ERROR messages are
                        displayed, and no reports are done by default
    --ignore=<file>[,<file>...]
                        Add files or directories to the blacklist. They should
                        be base names, not paths. [current: CVS]

  Commands:
    --help-msg=<msg-id>
                        Display a help message for the given message id and
                        exit. The value may be a comma separated list of
                        message ids.
    --generate-rcfile   Generate a sample configuration file according to the
                        current configuration. You can put other options
                        before this one to get them in the generated
                        configuration.

  Messages control:
    -e <msg ids>, --enable=<msg ids>
                        Enable the message, report, category or checker with
                        the given id(s). You can either give multiple
                        identifier separated by comma (,) or put this option
                        multiple time.
    -d <msg ids>, --disable=<msg ids>
                        Disable the message, report, category or checker with
                        the given id(s). You can either give multiple
                        identifier separated by comma (,) or put this option
                        multiple time (only on the command line, not in the
                        configuration file where it should appear only once).

  Reports:
    -f <format>, --output-format=<format>
                        Set the output format. Available formats are text,
                        parseable, colorized, msvs (visual studio) and html
                        [current: text]
    -i <y_or_n>, --include-ids=<y_or_n>
                        Include message's id in output [current: no]
    -r <y_or_n>, --reports=<y_or_n>
                        Tells whether to display a full report or only the
                        messages [current: yes]

How-To-Repeat: cd /usr/ports/devel/pylint; make install
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2012-07-31 15:56:31 UTC
Responsible Changed
From-To: freebsd-bugs->freebsd-ports-bugs

ports PR.
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2012-07-31 15:57:07 UTC
Responsible Changed
From-To: freebsd-ports-bugs->mva

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 3 dfilter service freebsd_committer freebsd_triage 2012-08-02 08:39:15 UTC
Author: mva
Date: Thu Aug  2 07:39:04 2012
New Revision: 301857
URL: http://svn.freebsd.org/changeset/ports/301857

Log:
  - Fix file permissions for py2stdlib module
  
  PR:		ports/170290
  Reported by:	Garrett Cooper <yaneurabeya@gmail.com>

Modified:
  head/devel/py-astng/Makefile

Modified: head/devel/py-astng/Makefile
==============================================================================
--- head/devel/py-astng/Makefile	Thu Aug  2 07:24:39 2012	(r301856)
+++ head/devel/py-astng/Makefile	Thu Aug  2 07:39:04 2012	(r301857)
@@ -7,6 +7,7 @@
 
 PORTNAME=	astng
 PORTVERSION=	0.24.0
+PORTREVISION=	1
 CATEGORIES=	devel python
 MASTER_SITES=	LOGILAB
 PKGNAMEPREFIX=	${PYTHON_PKGNAMEPREFIX}
@@ -22,4 +23,9 @@ USE_PYTHON=		yes
 USE_PYDISTUTILS=	yes
 PYDISTUTILS_PKGNAME=	logilab-astng
 
-.include <bsd.port.mk>
+.include <bsd.port.pre.mk>
+
+post-patch:
+	@${CHMOD} a+r ${WRKSRC}/brain/py2stdlib.py
+
+.include <bsd.port.post.mk>
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 4 Marcus von Appen freebsd_committer freebsd_triage 2012-08-02 08:39:42 UTC
State Changed
From-To: open->closed

This has been fixed. Thanks!