ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Alexey Tourbin <at@altlinux.ru>
To: devel@lists.altlinux.org
Subject: [devel] xorg server API changes (was: [#20188] FAILED (try 5) xorg...)
Date: Wed, 17 Feb 2010 16:54:56 +0300
Message-ID: <20100217135456.GA8069@altlinux.org> (raw)
In-Reply-To: <20100217095254.GA13927@ssh.git.orion.altlinux.org>

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

On Wed, Feb 17, 2010 at 12:52:58PM +0300, Girar Builder robot wrote:
> http://git.altlinux.org/tasks/20188/task/log.5

> #85 build 0.3.2-alt2 from /people/shrek/packages/xorg-drv-dummy.git
> 2010-Feb-17 12:22:02 :: created pkg.tar for xorg-drv-dummy.git tag 0.3.2-alt2

> 2010-Feb-17 12:42:53 :: [i586] xorg-drv-dummy.git 0.3.2-alt2: build start
> 2010-Feb-17 12:43:38 :: [i586] xorg-drv-dummy.git 0.3.2-alt2: build OK
> 2010-Feb-17 12:48:26 :: build check OK

2shrek:  I'm lost here.  What you're trying to do is plain wrong.
ELF symbols check finds you a bunch of bugs, for free (by the way,
that might seem easy, but it isn't).  At least some of these bugs are
already classified as "crash bugs" - X.org server is going to crash,
possibly upon stratup.  If anything, you are expected to fix the bugs.
Instead, you're trying to circumvent the check by fooling the robot!

For example, xf86UDealy() function has been removed from the X.org server API:
http://git.altlinux.org/people/shrek/packages/xorg-server.git?a=commitdiff;h=9522ab9a
That is, calling xf86UDealy() from a video driver is going to crash the
server.  This function seems to be unused in modern drivers anyway, but
legacy drivers must be somehow adjusted - e.g. probably it would suffice
to replace xf86UDealy() with usleep().

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

       reply	other threads:[~2010-02-17 13:54 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-17 13:54 ` Alexey Tourbin [this message]
2010-02-17 14:08   ` [devel] xorg server API changes Anton Farygin
2010-02-17 14:12     ` Dmitry V. Levin
2010-02-17 14:19       ` Michael Shigorin
2010-02-17 14:23       ` Valery V. Inozemtsev
2010-02-17 14:40         ` Anton Farygin
2010-02-17 14:45           ` Valery V. Inozemtsev
2010-02-18  5:32       ` Шишков Евгений
2010-02-18  6:00         ` Motsyo Gennadi aka Drool
2010-02-18 10:52           ` Max Ivanov
2010-02-18 12:17             ` Шишков Евгений
2010-02-18 12:36               ` Andrey Rahmatullin
2010-02-17 14:20     ` Alexey Tourbin

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=20100217135456.GA8069@altlinux.org \
    --to=at@altlinux.ru \
    --cc=devel@lists.altlinux.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

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git