Linux console tools development discussion
 help / color / mirror / Atom feed
* [kbd] [Fwd: Re: [Pkg-kbd-devel] kbd -b needlessly requires VT access]
@ 2016-08-29 15:17 Andy Valencia
  2016-09-06 15:49 ` Alexey Gladkov
  0 siblings, 1 reply; 2+ messages in thread
From: Andy Valencia @ 2016-08-29 15:17 UTC (permalink / raw)
  To: kbd

At the request of Andreas, I am forwarding this bug I reported on Debian.
Briefly, loadkeys in -b mode is just a file utility, converting text
format to binary.  And yet the code is structured so that a console
FD must be opened, needlessly requiring the file conversion to be run
in a console environment.

Thanks,
Andy Valencia


---------------------------- Original Message ----------------------------
Subject: Re: [Pkg-kbd-devel] kbd -b needlessly requires VT access
From:    "Andreas Henriksson" <andreas@fatal.se>
Date:    Sun, August 28, 2016 4:59 am
--------------------------------------------------------------------------

Hello Andy Valencia.

On Fri, Aug 26, 2016 at 08:54:53AM -0700, Andy Valencia wrote:
> When generating a binary map using "loadkeys -b" to target a smaller
> device, the command fails unless run in an environment where it can
> access a console device for itself.  This is due to its unconditional
> use of getfd(), even when all the command is going to do is write
> out a binary version of its input file.

I see that the fd returned from getfd is only conditionally used, but
the call to getfd will result in exit(1) being called if getfd is
about to fail. This indeed sounds suboptimal.
Not sure if moving the getfd call to where fd is used, or if guarding
the early call to getfd with the same guards that will make use
of getfd, is the best option.

Could you please try to contact the upstream development list about
this issue?

https://lists.altlinux.org/mailman/listinfo/kbd

Regards,
Andreas Henriksson




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-09-06 15:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-29 15:17 [kbd] [Fwd: Re: [Pkg-kbd-devel] kbd -b needlessly requires VT access] Andy Valencia
2016-09-06 15:49 ` Alexey Gladkov

Linux console tools development discussion

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/kbd/0 kbd/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 kbd kbd/ http://lore.altlinux.org/kbd \
		kbd@lists.altlinux.org kbd@lists.altlinux.ru kbd@lists.altlinux.com
	public-inbox-index kbd

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


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