lp:xf86-input-wacom

Created by Jason Gerecke and last modified
Get this branch:
bzr branch lp:xf86-input-wacom

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
VCS imports
Project:
Xf86 Input Wacom
Status:
Mature

Import details

Import Status: Failed

This branch is an import of the HEAD branch of the Git repository at https://github.com/linuxwacom/xf86-input-wacom.git.

The import has been suspended because it failed 5 or more times in succession.

Last successful import was .

Import started on juju-98ee42-prod-launchpad-codeimport-4 and finished taking 10 seconds — see the log
Import started on juju-98ee42-prod-launchpad-codeimport-3 and finished taking 10 seconds — see the log
Import started on juju-98ee42-prod-launchpad-codeimport-2 and finished taking 5 seconds — see the log
Import started on juju-98ee42-prod-launchpad-codeimport-2 and finished taking 5 seconds — see the log

Recent revisions

1646. By James Pearson <email address hidden>

Running 'xsetwacom get $id Serial' gives:

  Unknown parameter name 'Serial'.

'Serial' should be in the deprecated parameter list - also 'GetTabletID'
is defined twice in the same list

Signed-off-by: James Pearson <email address hidden>
Reviewed-by: Jason Gerecke <email address hidden>
Signed-off-by: Peter Hutterer <email address hidden>

1645. By Jason Gerecke

travis: Hotfix Coverity's Travis integration patch

Coverity has been triggering build errors for quite a while now, simply
because their server returns a different success code after uploading
than their script demands. We've notified them about this bug, but
nothing seems to be happening. Lets just patch the broken code and move
on with life.

Signed-off-by: Jason Gerecke <email address hidden>
Acked-by: Ping Cheng <email address hidden>

1644. By Ping Cheng <email address hidden>

xsetwacom: update man page for button mapping

Mapping a button to 0 is to disable it in X.

Signed-off-by: Ping Cheng <email address hidden>

1643. By Aaron Skomra

release.sh: bugfixes

1. Use correct filename variable (tarbz2 not tarball).
2. Insert spaces into heredoc to make links work for Gmail users.
3. The script greps the module url to see if it contains an
   approved repository. The first update to the release script
   contained a list of many approved repositories. Subsequent updates
   during the review process had these extra repositories removed.
   (git update-index --assume-unchanged release.sh can be used to
   temporarily use a different repository.) That second change
   removed
           -e "linuxwacom/.*" (Github)
   instead of
           -e "/linuxwacom/.*" (Sourceforge).
   Remove the slash which is not found in the Github path.
4. Remove the language about xorg, mesa, etc. from error message.
5. Correct release.sh script documentation. The release.sh script no
   longer has a `--user` option. The username is now provided as a
   required argument of the `--github` and `--sourceforge` options.
   Note that when using `--github`, a personal access token may need
   to be appended to the username with a colon if two-factor auth is
   used.

Fixes: 4c17c6a0dd ("Update release.sh script from Xorg") in the tags.
Fixes: 32c065ae02 ("add Github to release.sh script")
Signed-off-by: Aaron Armstrong Skomra <email address hidden>
Signed-off-by: Jason Gerecke <email address hidden>

1642. By Jason Gerecke

wacom 0.36.1

Signed-off-by: Jason Gerecke <email address hidden>

1641. By Aaron Skomra

Update README to reflect the move to Github.

Signed-off-by: Aaron Armstrong Skomra <email address hidden>
Reviewed-by: Peter Hutterer <email address hidden>

1640. By Jason Gerecke

Fail the Coverity build if their script cannot be downloaded

If curl encounters an error trying to download the Coverity build script
(as seems to be happening at the moment due to SSL issues), we will not
be notified since the pipe swallows the error code. This is not ideal, so
this commit updates the Travis build steps to use a slightly more contrived
construction which will get and then execute the script as independent steps.
Also, to ensure curl returns an error code even in less-severe cases (e.g. a
basic 404 "Not Found") we add the '-f' option.

Signed-off-by: Jason Gerecke <email address hidden>
Reviewed-by: Peter Hutterer <email address hidden>

1639. By Jason Gerecke

Check both valid and working state when finding new channels

The usbChooseChannel function is responsible for finding the channel
that should be used to contain a sequence of events for a particular
tool/contact. If a tool/contact is already using a particular channel
(i.e., the last event in the channel has the correct device type, serial
number, and is in proximity) then usbChooseChannel will return that
channel. Otherwise, usbChooseChannel will try to find a free channel
(i.e., one which is out of proximity) to be used to store the tool's
sequence of events.

The existing logic has a subtle bug that may result in a malfunction
when a specific condition occurs. In particular, if a multitouch report
from the kernel contains an existing contact going up followed by a
brand new contact, then usbChooseChannel will see the channel associated
with the former contact as free and begin using it to store data for the
new contact. As a result, the "up" event is never sent and can lead to
further strange touch behavior (including the effective disabling of
touch input).

To fix this, we modify usbChooseChannel to consider the "valid" state as
well as the working state when finding a new channel. A channel should
not be considered free if its valid state is still in prox, since this
indicates that the channel has *just* left prox. Note that we cannot
rely on only checking the valid state since this fails in the opposite
way: usbChooseChannel would not notice when a channel has *just* entered
prox and would overwrite its contents.

It isn't obvious wether it is also necessary to check the valid state
for the other two cases in this function (finding an existing channel
and forcefully clearing channels when out of space). The existing code
seems to work, however, so we will leave it as-is.

Ref: https://sourceforge.net/p/linuxwacom/bugs/335/?limit=25&page=1#73fe
Ref: https://sourceforge.net/p/linuxwacom/bugs/339/#f940
Ref: https://sourceforge.net/p/linuxwacom/bugs/349/
Signed-off-by: Jason Gerecke <email address hidden>
Reviewed-by: Peter Hutterer <email address hidden>

1638. By Daniel Ferguson

Fix tilt-y being overridden with tilt-x

Refactoring in (527fa95c29) introduced a typo in wcmFilterCoord which
resulted in the value of tilt x being assigned to tilt y.

Ref: https://github.com/linuxwacom/xf86-input-wacom/issues/11
Fixes: 527fa95c29 ("Refactor coordinate averaging to seperate function")
Signed-off-by: Daniel Ferguson <email address hidden>
Signed-off-by: Jason Gerecke <email address hidden>
Reviewed-by: Peter Hutterer <email address hidden>

1637. By Ping Cheng

Support DTK-2451 and DTH-2452 on older kernels

Signed-off-by: Ping Cheng <email address hidden>
Signed-off-by: Peter Hutterer <email address hidden>

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
This branch contains Public information 
Everyone can see this information.

Subscribers