Permalink
Browse files

Make fedmsg-config *not* a setuptools script.

Fixes #72.
  • Loading branch information...
1 parent eb12c2e commit a7860d19609c0bf6c0065b6cc95f55e478a98fc4 @ralphbean committed Nov 15, 2012
Showing with 23 additions and 1 deletion.
  1. +2 −0 MANIFEST.in
  2. +4 −0 scripts/fedmsg-config
  3. +17 −1 setup.py
View
@@ -8,3 +8,5 @@ recursive-include extras *
recursive-include doc *
recursive-include fedmsg.d *.py
recursive-include init.d *
+
+graft scripts
View
@@ -0,0 +1,4 @@
+#!/usr/bin/env python
+
+import fedmsg.commands.config
+fedmsg.commands.config.config()
View
@@ -90,14 +90,30 @@
],
include_package_data=True,
zip_safe=False,
+ scripts = [
+ # This is separate from the other console scripts just for efficiency's
+ # sake. It gets called over and over and over again by our mediawiki
+ # plugin/mod_php. By making it *not* a setuptools console_script it
+ # does a lot less IO work to stand up.
+
+ # Before:
+ # $ strace fedmsg-config 2>&1 | wc -l
+ # 34843
+
+ # After:
+ # $ strace fedmsg-config 2>&1 | wc -l
+ # 13288
+
+ 'scripts/fedmsg-config',
+ ],
entry_points={
'console_scripts': [
"fedmsg-logger=fedmsg.commands.logger:logger",
"fedmsg-tail=fedmsg.commands.tail:tail",
"fedmsg-hub=fedmsg.commands.hub:hub",
"fedmsg-relay=fedmsg.commands.relay:relay",
"fedmsg-gateway=fedmsg.commands.gateway:gateway",
- "fedmsg-config=fedmsg.commands.config:config",
+ #"fedmsg-config=fedmsg.commands.config:config",
"fedmsg-irc=fedmsg.commands.ircbot:ircbot",
"fedmsg-collectd=fedmsg.commands.collectd:collectd",
"fedmsg-tweet=fedmsg.commands.tweet:tweet",

0 comments on commit a7860d1

Please sign in to comment.