2. Hstox test

[iphydf]

This change is

[GrayHatter]

Reviewed 4 of 5 files at r1.
Review status: 4 of 5 files reviewed at latest revision, 1 unresolved discussion.

---

other/travis/hstox-install, line 19 [r1] (raw file):

# The reason we move source files instead of the whole git root directory is
# that the travis scripts (like this one) need to remain in place.
mv toxcore ../hstox/test/toxcore/toxcore

mv not cp -a ?

---

Comments from Reviewable

[iphydf]

Review status: 4 of 5 files reviewed at latest revision, 1 unresolved discussion.

---

other/travis/hstox-install, line 19 [r1] (raw file):

Previously, GrayHatter (Gregory Mullen) wrote…

mv not cp -a ?

It's faster (O(1)) and there is no reason to keep it in its old place. I agree that the gains of avoiding a copy of 20ish files are minimal, but it doesn't add complexity. I can change it if you feel this is confusing.

---

Comments from Reviewable

[GrayHatter]

---

Reviewed 1 of 5 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

other/travis/hstox-install, line 19 [r1] (raw file):

Previously, iphydf wrote…

It's faster (O(1)) and there is no reason to keep it in its old place. I agree that the gains of avoiding a copy of 20ish files are minimal, but it doesn't add complexity. I can change it if you feel this is confusing.

LGTM

---

Comments from Reviewable