[OpenWrt-Devel] [PATCH] Add munin to /etc/services

2012-03-18 Thread Jonathan McCrohan
The muninlite postinst fails to add munin to /etc/services.

This patch removes this broken postinst and add an entry for munin in
/etc/services. This should have been done when the rest of the
/etc/services postinsts were being removed (changesets 26142-26149).

Signed-off-by: Jonathan McCrohan jmccro...@gmail.com

diff --git a/package/base-files/files/etc/services 
b/package/base-files/files/etc/services
index 10161eb..a12853e 100644
--- a/package/base-files/files/etc/services
+++ b/package/base-files/files/etc/services
@@ -124,6 +124,7 @@ iax 4569/tcp
 iax4569/udp
 mtn4691/tcp
 mtn4691/udp
+munin  4949/tcp
 sip5060/tcp
 sip5060/udp
 sip-tls5061/tcp


diff --git a/admin/muninlite/Makefile b/admin/muninlite/Makefile
index a55d3de..d241707 100644
--- a/admin/muninlite/Makefile
+++ b/admin/muninlite/Makefile
@@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=muninlite
 PKG_VERSION:=1.0.4
-PKG_RELEASE:=2
+PKG_RELEASE:=3
 
 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz
 PKG_SOURCE_URL:=@SF/$(PKG_NAME)
@@ -30,13 +30,6 @@ define Package/muninlite/Default/description
Munin is a monitoring system for Unix networks.
 endef
 
-define Package/muninlite/postinst
-#!/bin/sh
-grep -q '^munin[[:space:]]*4949/tcp' $${IPKG_INSTROOT}/etc/services 2/dev/null
-[ $$? -ne 0 ]  echo munin   4949/tcp  $${IPKG_INSTROOT}/etc/services
-exit 0
-endef
-
 define Package/muninlite/install
$(INSTALL_DIR) $(1)/usr/sbin/
$(INSTALL_BIN) $(PKG_BUILD_DIR)/munin-node $(1)/usr/sbin/



signature.asc
Description: OpenPGP digital signature
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] [OpenWrt-Commits] r30966 - trunk/target/linux/x86

2012-03-18 Thread Philip Prindeville
Why is this going into x86 and not x86/generic?


On 3/17/12 3:32 PM, openwrt-comm...@openwrt.org wrote:
 Author: ryd
 Date: 2012-03-17 22:32:08 +0100 (Sat, 17 Mar 2012)
 New Revision: 30966
 
 Modified:
trunk/target/linux/x86/config-default
 Log:
 Enable keyboard for x86
 
 This is required for qemu like descripted in
 here http://wiki.openwrt.org/doc/howto/qemu
 
 
 
 Modified: trunk/target/linux/x86/config-default
 ===
 --- trunk/target/linux/x86/config-default 2012-03-17 21:30:31 UTC (rev 
 30965)
 +++ trunk/target/linux/x86/config-default 2012-03-17 21:32:08 UTC (rev 
 30966)
 @@ -164,6 +164,8 @@
  # CONFIG_IBM_RTL is not set
  CONFIG_ILLEGAL_POINTER_VALUE=0
  CONFIG_INITRAMFS_SOURCE=
 +CONFIG_INPUT=y
 +CONFIG_INPUT_KEYBOARD=y
  CONFIG_INSTRUCTION_DECODER=y
  # CONFIG_IOMMU_API is not set
  # CONFIG_IOMMU_HELPER is not set
 @@ -190,6 +192,7 @@
  CONFIG_KERNEL_GZIP=y
  # CONFIG_KERNEL_LZMA is not set
  CONFIG_KEXEC=y
 +CONFIG_KEYBOARD_ATKBD=y
  CONFIG_KTIME_SCALAR=y
  # CONFIG_M386 is not set
  CONFIG_M486=y
 
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[OpenWrt-Devel] patch to enable building of uuencode applet for busybox

2012-03-18 Thread i iordanov
Hello,

The package luci-app-pbx-voicemail needs uuencode which is unavailable
in busybox. I need it in order to encode binary attachments
(voicemail) to send by email. Including uuencode will naturally enable
OpenWRT machines to send binary email attachments of other kinds as
well.

Enabling uuencode increases the size of busybox only slightly. Here is
the difference:

1) Busybox with uuencode disabled:
Sizes reported by ls -l and du -s /bin/busybox respectively:
428527
419 /bin/busybox

2) Busybox with uuencode enabled:
Sizes reported by ls -l and du -s /bin/busybox respectively:
429327
420 /bin/busybox

This is less than 1kb difference. Do you think that we can enable the
building of uuencode by default? I am including a patch (created with
svn diff in the root directory of the SVN repository) which is
probably not the correct format, and this is why I need your input to
tell me how to format it correctly for submission as an enhancement.
Also, any input on whether it will be accepted is very welcome.

Many thanks,
Iordan

-- 
The conscious mind has only one thread of execution.
Index: package/busybox/config/coreutils/Config.in
===
--- package/busybox/config/coreutils/Config.in	(revision 30964)
+++ package/busybox/config/coreutils/Config.in	(working copy)
@@ -831,7 +831,7 @@
 
 config BUSYBOX_CONFIG_UUENCODE
 	bool uuencode
-	default n
+	default y
 	help
 	  uuencode is used to uuencode a file.
 
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] [PATCH] parted disc partitioning utility

2012-03-18 Thread Jo-Philipp Wich
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

I now looked over the patch and have a couple of comments.
First of all, I added GPT support to the trunk kernel config a few days
ago, so it should be always there by now.

I don't like conditional compilation of kernel features triggered by a
userspace package, that usually implies a lot of headaches for packagers
and distributors down the road.

Some more comments inline.

 --- package/parted/Makefile   (revision 0)
 +++ package/parted/Makefile   (revision 0)
 @@ -0,0 +1,54 @@
 +#
 +# Copyright (C) 2012 OpenWrt.org
 +#
 +# This is free software, licensed under the GNU General Public License v2.
 +# See /LICENSE for more information.
 +#
 +include $(TOPDIR)/rules.mk
 +
 +PKG_NAME:=parted
 +PKG_VERSION:=2.4
 +PKG_RELEASE:=1
 +
 +PKG_BUILD_DIR:=$(BUILD_DIR)/parted-$(PKG_VERSION)

This defaults to $(BUILD_DIR)/$(PKG_NAME)-$(PKG_VERSION) so it should
be safe to remove.

 +PKG_SOURCE:=parted-$(PKG_VERSION).tar.gz
 +PKG_SOURCE_URL:=@GNU/parted
 +PKG_MD5SUM:=76a6457ea88447d79d50ca331069b19c
 +PKG_CAT:=zcat

The decompression method is derived from the PKG_SOURCE extension so it
can be dropped as well.

 +
 +include $(INCLUDE_DIR)/package.mk
 +
 +define Package/parted
 + SECTION:=utils
 + CATEGORY:=Utilities
 + SUBMENU:=disc
 + DEFAULT:=n

That would cause the package to be never built for releases and
snapshots, probably not what you want. I suggest to remove the kernel
config overrides and this default-off as well.

 + TITLE:=GNU Parted manipulates partition tables
 + URL:=http://www.gnu.org/software/parted/
 + DEPENDS:= +libuuid +libreadline +libncurses
 +endef
 +
 +define Package/parted/description
 + GNU Parted manipulates partition tables. This is useful for creating 
 + space for new operating systems, reorganizing disk usage, copying data 
 + on hard disks and disk imaging. The package contains a library, 
 + libparted, as well as well as a command-line frontend, parted, 
 + which can also be used in scripts.
 +endef
 +
 +define Package/parted/config
 + source $(SOURCE)/Config.in
 +endef
 +
 +define Build/Configure
 + $(call Build/Configure/Default, \
 +--enable-static \
 +)
 +endef

You can drop the entire section and add

  CONFIGURE_ARGS += --enable-static

instead.

 +
 +define Package/parted/install
 + $(INSTALL_DIR) $(1)/usr/sbin
 + $(INSTALL_BIN) $(PKG_BUILD_DIR)/parted/parted $(1)/usr/sbin/
 +endef
 +
 +$(eval $(call BuildPackage,parted))


~ Jow
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk9mVV8ACgkQdputYINPTPNFWwCgj2aaOElGSCwW+iLGw6xX+eci
Xe4AnRu9FN80EC8zAolgmYIJF9qzTCw/
=gF3J
-END PGP SIGNATURE-
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] patch to enable building of uuencode applet for busybox

2012-03-18 Thread Philip Prindeville
Why is voicemail encoding using uuencode and not base64???


On 3/18/12 3:23 PM, i iordanov wrote:
 Hello,
 
 The package luci-app-pbx-voicemail needs uuencode which is unavailable
 in busybox. I need it in order to encode binary attachments
 (voicemail) to send by email. Including uuencode will naturally enable
 OpenWRT machines to send binary email attachments of other kinds as
 well.
 
 Enabling uuencode increases the size of busybox only slightly. Here is
 the difference:
 
 1) Busybox with uuencode disabled:
 Sizes reported by ls -l and du -s /bin/busybox respectively:
 428527
 419 /bin/busybox
 
 2) Busybox with uuencode enabled:
 Sizes reported by ls -l and du -s /bin/busybox respectively:
 429327
 420 /bin/busybox
 
 This is less than 1kb difference. Do you think that we can enable the
 building of uuencode by default? I am including a patch (created with
 svn diff in the root directory of the SVN repository) which is
 probably not the correct format, and this is why I need your input to
 tell me how to format it correctly for submission as an enhancement.
 Also, any input on whether it will be accepted is very welcome.
 
 Many thanks,
 Iordan
 
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[OpenWrt-Devel] [PATCH] wavemon: Bump to v0.7.4

2012-03-18 Thread Jonathan McCrohan
Signed-off-by: Jonathan McCrohan jmccro...@gmail.com

---
 net/wavemon/Makefile |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/wavemon/Makefile b/net/wavemon/Makefile
index 14b099b..9724879 100644
--- a/net/wavemon/Makefile
+++ b/net/wavemon/Makefile
@@ -8,12 +8,12 @@
 include $(TOPDIR)/rules.mk

 PKG_NAME:=wavemon
-PKG_VERSION:=0.7.3
+PKG_VERSION:=0.7.4
 PKG_RELEASE:=1

 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.bz2
 PKG_SOURCE_URL:=http://eden-feed.erg.abdn.ac.uk/wavemon/stable-releases
-PKG_MD5SUM:=a96bad0359592ac56532067002b9e862
+PKG_MD5SUM:=7dc34d568dbaa1cb30c8cfc7ad0bb856

 PKG_INSTALL:=1

-- 
1.7.9.1



signature.asc
Description: OpenPGP digital signature
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel