All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephan Hoffmann <sho@relinux.de>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] gnuplot : new package
Date: Fri, 04 Jan 2013 18:44:15 +0100	[thread overview]
Message-ID: <50E714EF.4090102@relinux.de> (raw)
In-Reply-To: <20130104172643.72321640@skate>

Am 04.01.2013 17:26, schrieb Thomas Petazzoni:
> Dear Stephan Hoffmann,
>
> On Fri, 04 Jan 2013 16:32:20 +0100, Stephan Hoffmann wrote:
>
>>> +GNUPLOT_CONF_OPT = --without-x \
>> is there a special reason for this? Why not build with x, when X11 is
>> configured in buildroot?
> I guess because Anthony didn't need X support. It is not because a
> given software component can potentially support feature Y or Z that we
> require the package submitter to support all those features Y or Z. It
> would put way too much load on the initial package submitter.
Hello Thomas,

maybe my question was not clear enough. I just wanted to know if this
option is really required.
> Instead, we prefer having small packages that only support the features
> that have been tested by the submitter, and that carefully disables all
> the unsupported features. And then, as features are needed by other
> people, they can submit additional patches to make those additional
> features work.
O.K., I didn't realize that even when configure finds out that no X11 is
present --without-x is needed for this reason.

For the records: I made a quick test without X11 and it fails to compile
without this option.
> Anthony: a gnuplot patch without the X support is perfectly acceptable.
Of course, this is my opinion, too! I really like seeing it included!

Kind regards

Stephan
>
> Best regards,
>
> Thomas


-- 
reLinux     -    Stephan Hoffmann
Am Schmidtgrund 124    50765 K?ln
Tel. +49.221.95595-19    Fax: -64
www.reLinux.de     sho at reLinux.de

  reply	other threads:[~2013-01-04 17:44 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-04 14:53 [Buildroot] [PATCH] gnuplot : new package Anthony Viallard
2013-01-04 15:15 ` Thomas Petazzoni
2013-01-04 15:38   ` Viallard Anthony
2013-01-04 15:32 ` Stephan Hoffmann
2013-01-04 16:26   ` Thomas Petazzoni
2013-01-04 17:44     ` Stephan Hoffmann [this message]
2013-01-04 18:02       ` Thomas Petazzoni
2013-01-04 18:01 ` Stephan Hoffmann

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=50E714EF.4090102@relinux.de \
    --to=sho@relinux.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.