From: Bernhard Fischer <rep.dot.nop@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] rxvt-package for all x11-versions
Date: Tue, 28 Aug 2007 23:19:42 +0200 [thread overview]
Message-ID: <20070828211941.GB11697@aon.at> (raw)
In-Reply-To: <f90898420708280735q1852713eh4a6c1846fc8d827d@mail.gmail.com>
On Tue, Aug 28, 2007 at 04:35:48PM +0200, Simon Pasch wrote:
>This is my attempt to realize a rxvt-version for xorg, tinyx and the new x11r7.
>I tested it with xorg and x11r7. (Don't know if it works with tinyx,
>because the tinyx-build fails)
>
>By the way...Is the file rxvt_2.6.4-10.diff needed anymore?!
>
>diff -Naur buildroot/package/rxvt.old/Config.in buildroot/package/rxvt/Config.in
>--- buildroot/package/rxvt.old/Config.in 2007-08-28 09:15:23.000000000 +0200
>+++ buildroot/package/rxvt/Config.in 2007-08-28 15:19:17.000000000 +0200
>@@ -1,10 +1,8 @@
> config BR2_PACKAGE_RXVT
> bool "rxvt"
> default n
>- depends BR2_PACKAGE_XSERVER_xorg || BR2_PACKAGE_XSERVER_tinyx
>-
>+ depends BR2_PACKAGE_XSERVER_xorg || BR2_PACKAGE_XSERVER_tinyx ||
>BR2_PACKAGE_XSERVER_x11r7
> help
> A nice small colour vt102 X terminal emulator.
>- Does not build for X11R7 yet.
>
> http://www.rxvt.org/
>diff -Naur buildroot/package/rxvt.old/rxvt.mk buildroot/package/rxvt/rxvt.mk
>--- buildroot/package/rxvt.old/rxvt.mk 2007-08-28 09:15:23.000000000 +0200
>+++ buildroot/package/rxvt/rxvt.mk 2007-08-28 16:17:30.000000000 +0200
>@@ -27,6 +27,18 @@
> RXVT_DIR:=$(BUILD_DIR)/rxvt-$(RXVT_VERSION)
> RXVT_BINARY:=$(RXVT_DIR)/src/rxvt
>
>+ifeq ($(strip $(BR2_PACKAGE_XSERVER_xorg)),y)
>+RXVT_PREFIX:=/usr/X11R6
>+endif
>+
>+ifeq ($(strip $(BR2_PACKAGE_XSERVER_tinyx)),y)
>+RXVT_PREFIX:=/usr/X11R6
>+endif
>+
>+ifeq ($(strip $(BR2_PACKAGE_XSERVER_x11r7)),y)
>+RXVT_PREFIX:=/usr
>+endif
>+
> $(DL_DIR)/$(RXVT_SOURCE):
> $(WGET) -P $(DL_DIR) $(RXVT_SITE)/$(RXVT_SOURCE)
>
>@@ -46,11 +58,11 @@
> --target=$(GNU_TARGET_NAME) \
> --host=$(GNU_TARGET_NAME) \
> --build=$(GNU_HOST_NAME) \
>- --prefix=/usr/X11R6 \
>+ --prefix=$(RXVT_PREFIX) \
Given that the filesystem standard nowadays seem to mandate that all X11
apps should go into /usr, it would be way better to add one central
config BR2_X11_PREFIX
string "/usr/X11R6" if BR2_PACKAGE_XSERVER_xorg || BR2_PACKAGE_XSERVER_tinyx
string "/usr" if !BR2_PACKAGE_XSERVER_xorg && !BR2_PACKAGE_XSERVER_tinyx
in package/Makefile.in, below the XSERVER settings.
Peronally i don't like that new convention to put the X11 bloat^Wstuff
into /usr, but that is not relevant here.
Please update accordingly and resend. TIA,
next prev parent reply other threads:[~2007-08-28 21:19 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-28 14:35 [Buildroot] rxvt-package for all x11-versions Simon Pasch
2007-08-28 21:19 ` Bernhard Fischer [this message]
2007-08-28 22:09 ` Cristian Ionescu-Idbohrn
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070828211941.GB11697@aon.at \
--to=rep.dot.nop@gmail.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox