Ticket #20918 (closed submission: fixed)
New port: cppcheck
| Reported by: | alek.silverstone@… | Owned by: | macports-tickets@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | Cc: | jpo@…, paj-macports@…, raimue@…, snc@… | |
| Port: | cppcheck |
Description
Cppcheck - A tool for static C/C++ code analysis
http://astyle.sourceforge.net/
http://sourceforge.net/apps/mediawiki/cppcheck/index.php?title=Main_Page#Getting_the_source_code
Attachments
Change History
comment:4 Changed 3 years ago by jpo@…
In order to compile the cppcheck command line tool in Leopard (10.5.8 with gcc 4.0.1) I had to disable the gcc -Wlogical-op flag
--- Makefile.orig 2010-03-09 18:07:43.000000000 +0000
+++ Makefile 2010-04-04 20:50:04.000000000 +0100
@@ -1,4 +1,4 @@
-CXXFLAGS=-Wall -Wextra -pedantic -Wfloat-equal -Wcast-qual -Wlogical-op -O2 -DNDEBUG
+CXXFLAGS=-Wall -Wextra -pedantic -Wfloat-equal -Wcast-qual -O2 -DNDEBUG
CXX=g++
BIN=${DESTDIR}/usr/bin
otherwise the build process would terminate with the following error message:
cc1plus: error: unrecognized command line option "-Wlogical-op"
comment:5 Changed 3 years ago by jpo@…
The man page and the pdf manual creation also need to be tweaked:
Makefile: Use /opt/local/share/xsl/docbook-xsl/manpages/docbook.xsl instead.
man/buildman.sh: Use /opt/local/share/xsl/docbook-xsl/xhtml/docbook.xsl and /opt/local/share/xsl/docbook-xsl/fo/docbook.xsl instead.
Changed 3 years ago by paj-macports@…
Initial version of Portfile for v1.44. Patch required.
Changed 3 years ago by paj-macports@…
- Attachment patch-Makefile.diff added
Patch to Makefile for correct binary installation.
comment:7 Changed 3 years ago by paj-macports@…
Have attached files to add cppcheck to macports.
port lint --nitpick
Returns no warnings nor errors.
Am happy to maintain this port.
comment:8 Changed 3 years ago by macsforever2000@…
- Keywords new request removed
- Type changed from request to submission
Changed 3 years ago by paj-macports@…
- Attachment Portfile.2 added
Added Portfile for v1.45 (no patch required)
comment:9 Changed 3 years ago by raimue@…
- Cc raimue@… added
The description field should be short, about one sentence to say what the purpose of the port is.
The port is not UsingTheRightCompiler, add build.args CXX=${configure.cxx} to the Portfile.
Changed 3 years ago by paj-macports@…
- Attachment Portfile.3 added
Fixed Short Description and Compiler, as per raimue's feedback.
comment:10 Changed 3 years ago by paj-macports@…
Fixed. Thanks for the feedback and the link.
comment:11 Changed 3 years ago by raimue@…
Here is an update which includes the man page. I also re-ordered some of the options to be in our typical Portfile style (order of execution: configure, build, (test), destroot).
comment:12 Changed 3 years ago by paj-macports@…
Thanks for adding the man page, raimue. I was still working out how to do that!
comment:14 Changed 3 years ago by paj-macports@…
I'm happy for it to go in.
comment:15 Changed 3 years ago by snc@…
- Status changed from new to closed
- Resolution set to fixed
Committed in r72324.
Cc Me!