linux-can.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marc Kleine-Budde <mkl@pengutronix.de>
To: Jan-Niklas Meier <dschanoeh@googlemail.com>
Cc: "linux-can@vger.kernel.org" <linux-can@vger.kernel.org>
Subject: Re: [Gitorious] Activity: dschanoeh updated merge request for...
Date: Mon, 05 Dec 2011 11:28:54 +0100	[thread overview]
Message-ID: <4EDC9CE6.7000705@pengutronix.de> (raw)
In-Reply-To: <CAEuXk934A1Vu_3bZ_kyhxO_XJiqoLOY0zuSRxxNxpmLXPjmRCg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2827 bytes --]

On 12/04/2011 10:57 AM, Jan-Niklas Meier wrote:
>>> To be able to compile a subset of the tools I checked if the isotp.h and
>>> gw.h are present in the include directory (I don't have them both) and only
>>> compile the corresponding utils if the headers are present. This way it
>>> should still be possible to compile subsets of the tools on machines with
>>> old kernels and not ending up with tools you can not use afterwards.
>>
>> What about the external kernel modules use case?
> 
> Thank you very much for the clarification. Now I understand why you
> did this. Still if your kernel's CAN API is much older than the
> headers included in the current can-utils the programs will compile
> fine but it is possible that some of them will fail at runtime right?

Yes. If the kernel doesn't implement the features your userspace program
needs.

> Method signatures could have changed or some methods are not present
> in the old kernel. For example using a program with some specific
> parameters could lead to strange behavior for the user. Is that true
> or am I missing something?

Yes - you're missing the policy. The Kernel-Userspace interface must not
be changed in an incompatible way, at least once an interface is in the
mainline kernel. This includes not only the API but also the ABI (the
binary interface) to the kernel. This rule is (next to) 100% hard for
system calls, ioctls, etc... modern things like the sysfs tend to change
sometimes.

For example, the isotp programs will fail with "Protocol not supported"
during socket creation.

>>> I tried to rewrite the autoconf.ac because there are macros I'm having

[...]

>> That might be the problem, don't run autoconf, run $(./autogen.sh) instead.
> 
> \o/ Yay! That's the hint I needed. I didn't know that there is a
> different way to generate a ./configure that autoconf.

[...]

> I did the same and it works fine with both HEADs of your master and
> master-squashed branches. Still I get some warnings during autoreconf:
> 
> configure.ac:22: warning: The macro `AC_LIBTOOL_WIN32_DLL' is obsolete.
> configure.ac:22: You should run autoupdate.
> [...]
> configure.ac:24: warning: The macro `AC_PROG_LIBTOOL' is obsolete.
> configure.ac:24: You should run autoupdate.

These are deprecation warnings from the autotools. IIRC these come from
libtool-2.x, but you need them for pre pre 2.x to work. Debian lenny
(this is old-stable, isn't it?) uses still libtool-1.5. Maybe some old
redhat versions, too.

cheers, Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 262 bytes --]

  reply	other threads:[~2011-12-05 10:29 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20111129121452.B4F9F7590F@steelheart.shortcut.kunder.linpro.no>
2011-11-29 14:08 ` [Gitorious] Activity: dschanoeh updated merge request for Marc Kleine-Budde
     [not found]   ` <CAEuXk91Bh+Z0yTqQebmW-EmrU59c-dFz_zKj8ky0ssgFBy23bQ@mail.gmail.com>
2011-12-03 18:54     ` Marc Kleine-Budde
2011-12-04  9:57       ` Jan-Niklas Meier
2011-12-05 10:28         ` Marc Kleine-Budde [this message]
2011-12-05 10:30         ` Marc Kleine-Budde
2011-12-05 17:59           ` Jan-Niklas Meier

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=4EDC9CE6.7000705@pengutronix.de \
    --to=mkl@pengutronix.de \
    --cc=dschanoeh@googlemail.com \
    --cc=linux-can@vger.kernel.org \
    /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;
as well as URLs for NNTP newsgroup(s).