Release 2.5.5

Signed-off-by: Chris Radek <chris@timeguy.com>

increase stepgen to 16
Signed-off-by: John Thornton <jthornton@gnipsel.com>
(cherry picked from commit 61ebd8221eedad3d22b9e679c7857574b10c57d4)

Docs: update location of iso files
some people with old hardware may need to restore with older versions
of EMC2.

Signed-off-by: John Thornton <bjt128@gmail.com>

Fix pin-1-in-not

... which apparently nobody has ever used

Signed-off-by: Chris Radek <chris@timeguy.com>

interp: fix an old bug in canned cycle preliminary & in-between moves

The rules (for the XY plane, to simplify the terminology) are as follows:

1. (Preliminary) if OLD_Z is below R, rapid up to R
2. (In-between) rapid to XY location of next cycle
3. (In-between) rapid down to R, if needed
4. Perform cycle (leaves controlled point at CLEAR_Z)
5. If there are more cycles to do, goto 2.

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

interp: comment the "rapid up to R if needed" code

This adds a comment that explains the previous bugfix commit (0b552aa5).

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

Merge remote-tracking branch 'origin/v2.4_branch' into v2.5_branch

g81 test: add repeat cycles

This commit adds coverage of L-word repeats on G81 drill cycles.

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

fix canned cycles where old Z is below retract plane

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

update g81 tests for new init/shutdown Canon calls in 2.5

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

Merge remote-tracking branch 'origin/v2.4_branch' into v2.5_branch

Conflicts:
scripts/platform-is-supported (picked the version in 2.5)

tests: add an interpreter test of G81

This uses SAI to run G81 in different ways and verifies the resulting
Canon calls.

Tests G81 in G17, G18, and G19.

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

add platform-is-supported script in 2.4

This script lets the buildbot know what platforms to build the branch on.
2.4 doesn't build on newer distros.

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

packaging: dont build 2.5 on debian jessie

Jessie uses tcl 8.6, which is too new for 2.5.

Signed-off-by: Sebastian Kuzminsky <seb@highlab.com>

Fix a dubious type cast, from machinekit dac83399

Signed-off-by: Chris Radek <chris@timeguy.com>

hal/library/hal_link: fix fatal memory corruption bug on linking pin to a signal

Original two log messages:

see also https://github.com/machinekit/machinekit/issues/524

hal/unlink_pin: assure pin value on unlink is reset properly

This was introduced in 51a0afc76, and while not fatal,
this line

  *(hal_data_u *)dummy_addr = *sig_data_addr;

potentially restores more bytes from the signal than intended
(eg on a bit pin it will still store sizeof(double) bytes)
which may cause dummysig to hold confusing values when
inspected with a debugger

I (cradek) had to massage these commits somewhat to apply after mk's
refactoring etc.

My how-to-reproduce case:

halcmd: newsig s1 u32
halcmd: newsig s2 u32
halcmd: newsig s3 u32
halcmd: sets s1 0xaaaaaaaa
halcmd: sets s3 0x55555555
halcmd: show sig
Signals:
Type          Value  Name     (linked to)
u32      0xAAAAAAAA  s1
u32      0x00000000  s2
u32      0x55555555  s3

halcmd: net s2 => conv-u32-s32.0.in
halcmd: show sig
Signals:
Type          Value  Name     (linked to)
u32      0xAAAAAAAA  s1
u32      0x00000000  s2
 ==> conv-u32-s32.0.in
u32      0x00000000  s3

Notice s3 is overwritten.

Signed-off-by: Chris Radek <chris@timeguy.com>

Fix shift-jog speed being too slow on inch configs displaying mm

Fixes the second part of bug #397

Signed-off-by: Chris Radek <chris@timeguy.com>

Fix UVW jogs being too fast by 25.4x, on inch configs displaying mm

Signed-off-by: Chris Radek <chris@timeguy.com>

Merge remote-tracking branch 'jepler/signed-off-by' into v2.5_branch

Merge remote-tracking branch 'origin/v2.5_branch' into jepler/signed-off-by

Explicitly give the right build dependency for insmod/rmmod/etc

(cherry picked from commit 39e36ebd8fa5530ba87e162c17bac2c5ce57621f)

Conflicts:

debian/configure
debian/control.in

Fail on missing programs, instead of generating bogus scripts

(cherry picked from commit 0b5fba46aba7b35c8c3bcb80de43dec0ba8e97da)

These are needed for building and running on all platforms

(cherry picked from commit 3b03cefb49d964adaa9a83dc0f1775c4fdb97770)

Conflicts:
debian/control.in

deb: provide build-arch and build-indep targets

This is required by Debian Policy Manual section 4.9:

    https://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules

French doc update: fix startup code example

Signed-off-by: Francis Tisserant <tissf@free.fr>

DCO: Customize for LinuxCNC

.. noting that we require all submitted code to be licensed
compatibly with the GPLv2 with "or later" clause

Signed-off-by: Jeff Epler <jepler@unpythonic.net>

DCO: Move docs to the right location for linuxcnc

Signed-off-by: Jeff Epler <jepler@unpythonic.net>

docs: remove some broken asciidoc markup

(Variables) looks like an typoed attempt at creating an asciidoc anchor,
but it's not valid asciidoc anchor syntax so it doesnt work, and it
passes through into the generated docs, where it makes no sense.

None of the other asciidocs tries to use an anchor named Variables,
so just remove it.

Docs: fix startup code example.
g1 without a feed rate is an error and is a bad example for the startup code
thanks to archivist for spotting this
Signed-off-by: John Thornton <jthornton@gnipsel.com>

pncconf -fix error with firmware with more then 5 sserial channels

Pncconf can only display 5 channels but the code checked for 8
in certain circumstances.

French doc update: fix float value with old data

Signed-off-by: Francis Tisserant <tissf@free.fr>

French translation update

Signed-off-by: Francis Tisserant <tissf@free.fr>

pncconf -fix an icompatibility between MESA XMLs and linuxcnc XMLs

Linuxcnc used '???' to designate no secondary function, just GPIO,
where MESA use 'not used'. now both work
This made some GPIO pind not show in pncconf

Docs: try to word synopsis to be a bit clearer in the description

Signed-off-by: John Thornton <jthornton@gnipsel.com>

Docs: fix float value with old data

Signed-off-by: John Thornton <jthornton@gnipsel.com>

French doc update: Document G5,G5.1,G5.2,G5.3 NURBS/spline commands

Signed-off-by: Francis Tisserant <tissf@free.fr>

French doc update and cleaning

Signed-off-by: Francis Tisserant <tissf@free.fr>

linuxcnctop: fix the crawling scrollbar in many cases

.. by only changing the necessary part of the text, the 'crawl' is usually
avoided.  It might still occur in cases when a changing value is one that
is split across more than two lines (at least this is the pattern I
believed I saw when working on this problem).

It looks and smells like a Tk bug but oh well

Document G5, G5.1, G5.2, G5.3 NURBS/spline commands

This should close bug #369.

NURBS: fix biarc G1 continuity

The center of the second half of each biarc was wrong.

Clear saved state from previous NURBSes to avoid bogus motion

Previously if an incorrect NURBS specification was given, such as when
there aren't enough control points, an error would be generated, but a
second load would succeed because there were leftover control points
from last time.  This caused weird behavior such as every second program
load succeeding (when it should have failed every time), and then of
course giving bogus motion when run.

Docs: add link

Signed-off-by: John Thornton <jthornton@gnipsel.com>

Docs: add info on the servo axis calibration assistant

Signed-off-by: John Thornton <jthornton@gnipsel.com>

tests: verify that comp rejects .comp files whose names dont match the component name

comp: reject .comp files whose names dont match the component name

Docs: add info about file naming

Signed-off-by: John Thornton <jthornton@gnipsel.com>

manpages -update pages on classicladder

missing floats and pinnames were wrong for s32 pins

hal: fix minor bug in allocating hal memory for HAL_S32 and HAL_U32 signals

Signed-off-by: Michael Geszkiewicz <micges@wp.pl>

hal/link/unlink: revise semantics of hal_link()/hal_unlink():

old behavior: a pin's default value would be overwritten by a signal value
on link, effectively making the default pin value invisible outside the
component, and potentially causing a jump in the pin's value on link.

an unlink would re-apply the pin's hidden default value regardless of the
signal value, also causing a potential jump in the pin's value on unlink.

new behavior: the very first link of a signal to a pin will make the
signal inherit the pin's value. On unlink, the pin's default value will
be set from the current signal value.

The result is:
- no sudden jumps in pin values as a result of link/unlink
- it is now possible to access a pin's default value from other pins
- it is possible to have the changed default value retained on unlink

Change of behavior: Code relying on signal values defaulting to zero
post-linking should be revised to consider:

So far, a new signal value defaulted to 0/0.0/false regardless of
the pin's values linked to it.

Now, the signal will inherit the first linked pin's value.

ref: [Emc-developers] PID bidirectional pins. Mar 16 2014

docs: fix hm2 encoder.rawcounts pin name in manpage

The hm2 driver exports pins called "hm2_BLAH.X.encoder.YY.rawcounts",
but the manpage incorrectly called them ".rawcount".  Fix the manpage
to match the existing pin name.

comp: remove an unused and confusing global variable

This commit removes an unused global variable named __comp_state, from
.c files generated by the comp(1) tool.  The global was never set,
so it never served any actual purpose.

The functions declared by "function" declarations in the .comp file,
and the optional EXTRA_SETUP() function, all take an argument which is
also named __comp_state.  Thus, in these functions, the global was masked
by the function argument of the same name.  The __comp_state variable
is used by the convenience defines to access the hal pins and params of
the component instance.  The functions all worked fine and accessed the
hal pins via their argument, just like they should.

The problem was that the global made the convenience defines compile-time
valid everywhere in the .comp, but at run-time any non-"function",
non-EXTRA_SETUP code would access the uninitialized global and crash.

We're better off without it.

biquad: fix a crash-when-enabled bug

This fixes a bug in the biquad comp that would cause a crash with a NULL
pointer dereference immediately upon the user setting the .enable HAL
pin to True.  Thanks to forum user gaston48 for the bug report.

The biquad comp exports a hal function called "_" in the .comp and .c
files, called "biquad.0" in hal.  This function calls a function internal
to the biquad component (not exported to hal, not handled by comp(1))
called "Biquad_CalcCoeffs".  The Biquad_CalcCoeffs function accesses the
comp instance's hal pins and params, but this behavior is not supported
by the comp(1) tool.

Only functions handled by the comp tool can access the comp instance's hal
pins and params.  That means functions declared with a .comp "function"
declaration, and the EXTRA_SETUP() function.

This commit moves the Biquad_CalcCoeffs code from its separate function
into the main "_" aka "biquad.0" function, this avoiding the disallowed
"access hal from a called function" problem.

add a very basic test of the biquad component

docs: fix up some comp docs that had rotted a bit

The docs referred to an old name of an internal data structure.  The code
changed in a7a7fd39a but i forgot to update the docs.

The new name includes the underscore character "_", which is meaningful
to asciidoc.  This commit changes the markup delineating the name from
single-tick "'" to backtick "`", to tell asciidoc to pass through the
text without further substitution.  This also changes the rendering
from italic to monospace, which is arguably better anyway (though it's
somewhat inconsistent with the surrounding markup).

This commit also fixes a markup syntax blooper in the FOR_ALL_INSTS()
documentation.

French doc: cleaning

Signed-off-by: Francis Tisserant <tissf@free.fr>

French doc: add info about the direction arrow requirements

Signed-off-by: Francis Tisserant <tissf@free.fr>

Docs: fix spello

Signed-off-by: John Thornton <jthornton@gnipsel.com>

Docs: add info about the direction arrow requirements
Direction arrows must be a separate word with a space
between arrow and the pin name

Signed-off-by: John Thornton <jthornton@gnipsel.com>

Moves are way too short to make tones, if in mm mode

halsh: initialize stubs library

problem appeared for tclStubsPtr symbol with tcl8.6 in (x)ubuntu 14.04

edge.comp: fix output pulse width

The edge manpage claims that the output pulse width is controllable by
the .out_width_ns parameter, but there's a bug: the output pulse is one
thread period too long.

This commit fixes it.

edge.comp: fix .out-invert pin on first invocation

Before this commit, edge.*.out and .out_invert would both be 0 after the
first invocation of the component's function.  This commit fixes
.out_invert to always be the inverse of .out.

tests: add some tests of the edge component

Release 2.5.4

Changelog for 2.5.4

pncconf -fix off by one error on pin numbering of 5i25 boards

pncconf -honour 7i43 address changes

priviously always used default address.

added new configs for ppmc with encoder velocity estimation

edge.comp: eliminate false trigger upon startup

Fixing http://sourceforge.net/p/emc/bugs/346/

The edge component, edge.comp, starts with the last input value (last_in) 0, so
if you start the component, and the input is already 1, you will get a false
edge trigger.

Component: fix incorrect calculation of velocity tolerance percent
the velocity tolerance would end up being much too small as you increased
the value.

Signed-off-by: John Thornton <jthornton@gnipsel.com>

new pyvcp_widgets

this fixes http://sourceforge.net/p/emc/bugs/364/

Signed-off-by: Michael Haberler <git@mah.priv.at>

French doc: cleaning typo

Signed-off-by: Francis Tisserant <tissf@free.fr>

Ignore auto-generated manpages for two new comps

Add a component to access individual bits from a u32

Signed-off-by: Andy Pugh <andy@bodgesoc.org>

Add bitwise manipulations of u32.

Signed-off-by: Andy Pugh <andy@bodgesoc.org>

Comp: fix spelling error

Signed-off-by: John Thornton <jthornton@gnipsel.com>

French doc. update to follow John

Docs: expand a bit on the description to clear up confusion

Signed-off-by: Francis Tisserant <tissf@free.fr>

Docs: expand a bit on the description to clear up confusion

Signed-off-by: John Thornton <jthornton@gnipsel.com>

fix some broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

Merge branch 'ilowpass-wrap' into v2.5_branch

ilowpass: test for wraparound behavior

ilowpass: handle s32 overflow better

The conversion from double to int is undefined when the input operand
is out of range; in fact, gcc seems to return INT_MIN in this case.
By casting through __s64, and with -fwrapv specified, we get correct
signed integer wraparound until value exceeds 2^63.

Ask for -fwrapv, we depend on it

We want to be sure that e.g., (INT_MIN) - (INT_MAX) is -1, so that encoder
overflows work better.  The C standard says that the result of this
is implentation-defined, and gcc provides -fwrapv to ask for the common
"wraparound arithmetic" behavior.  -fwrapv is in all compilers we are
likely to encounter; the internet says it was added in gcc 3.3.

-fno-strict-overflow prevents the compiler from assuming things such as
`i + 10 > i` always being true.  It's not clear whether -frwapv effectively
implies -fno-strict-overflow (though in this specific case it should,
since -fwrapv defines some values of i for which the comparison is false),
so go ahead and specify it if available; the internet says it was
added in gcc 4.2.

fix another bad paste

Signed-off-by: Francis Tisserant <tissf@free.fr>

cleaning bad pastes

Signed-off-by: Francis Tisserant <tissf@free.fr>

cleaning & fix broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

fix broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

bad code delimiters previously entered by mystake

Signed-off-by: Francis Tisserant <tissf@free.fr>

Cleaning for a better use of titles

Signed-off-by: Francis Tisserant <tissf@free.fr>

fix some asciidoc warnings

Signed-off-by: Francis Tisserant <tissf@free.fr>

fix some broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

fix some broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

fix some broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

fix some broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

fix some broken links

Signed-off-by: Francis Tisserant <tissf@free.fr>

loadtool.ngc: provide options for g43 control

loadtool.ngc is a nc_files/ngcgui_lib/utilitysubs/ helper subroutine
for use with ngcgui_lib demonstration subroutines.

Add options for subroutine invocation:

   #<use_g43>
   #<h_for_g43>
   #<verbose>

to allow user to control behavior.

Also, allow toolno==0

Minor: use shorthand names for some call parameters since line length
limit is exceeded with additional items used.

loadtool.ngc uncondtionally apply G43

thanks to user pi.r for the report

ignore the new html docs stamp files

build system: fix parallel build of docs

Before this commit, each html file we built would copy its images from the
docs source directory to the docs html (target) directory.

Some of our html documents share images with other html documents, for
example many of the translations.

With 'make -j', and since all the html targets are built in parallel,
sometimes two make processes would try to copy the same image file at
the same time, resulting in failure.

This commit changes how images are handled.  All the html documents
are still handled in parallel, but the image copying is deferred until
after the html docs are all done, then all the images are copied by a
single thread.

deb/rules: dont use cpio when cp will do

Debian Wheezy doesn't come with cpio by default, this commit lets us
not add a build-dependency on it.

(cherry picked from commit 7a1c034150ab1d42f014f85600709e69ac194678)

Conflicts:
debian/rules.in