cvs commit: ports/databases Makefile ports/databases/py-flask-sqlalchemy Makefile distinfo pkg-descr pkg-plist

Alexey Dokuchaev danfe at FreeBSD.org
Sun Nov 28 15:26:24 UTC 2010


On Sat, Nov 27, 2010 at 02:01:36AM +0000, Philip M. Gollucci wrote:
> pgollucci    2010-11-27 02:01:35 UTC
> 
>   FreeBSD ports repository
> 
>   Modified files:
>     databases            Makefile 
>   Added files:
>     databases/py-flask-sqlalchemy Makefile distinfo pkg-descr 
>                                   pkg-plist 
>   Log:
>   Flask-SQLAlchemy adds support for SQLAlchemy to your Flask application.
>   
>   WWW:    http://pypi.python.org/pypi/Flask-SQLAlchemy
>   
>   PR:             ports/151626
>   Submitted by:   Olivier Duchateau <duchateau.olivier at gmail.com>

- Makefile is severely unindented
- PORTNAME is just disgustingly non-FreeBSDish [*]
- COMMENT should not be based on a verb instead of noun
- Missing tab in second RUN_DEPENDS line
- LICENSE should be better placed after COMMENT
- Port description contains style bug (tab after WWW:) and is identical to
  comment, ergo useless

[*] Original submission (which I've reviwed) did not contain this bug.
Why did you introduce it?

Please fix.

./danfe


More information about the cvs-all mailing list