doc: Split depends installation instructions per arch #14264

MarcoFalke commented at 12:48 AM on September 19, 2018: member

The current depends installation instructions fail on bionic with

E: Unable to locate package g++-4.8-aarch64-linux-gnu
E: Unable to locate package gcc-4.8-aarch64-linux-gnu
E: Unable to locate package g++-4.8-arm-linux-gnueabihf
E: Unable to locate package gcc-4.8-arm-linux-gnueabihf

Also, they fail due to missing dependencies make automake cmake pkg-config python3

Fix this by removing the explicit version and splitting them into common instructions and instructions per linux architecture.

doc: Split depends installation instructions per arch 0000009015

MarcoFalke added the label Docs on Sep 19, 2018

MarcoFalke commented at 12:56 AM on September 19, 2018: member

Fun fact:

git log 000000 -1
error: short SHA1 000000 is ambiguous
hint: The candidates are:
hint:   000000035 commit 2018-06-20 - Obsolete #!/bin/bash shebang
hint:   000000901 commit 2018-09-18 - doc: Split depends installation instructions per arch
fatal: ambiguous argument '000000': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

DrahtBot commented at 1:52 AM on September 19, 2018: member

No more conflicts as of last run.

MarcoFalke force-pushed on Sep 19, 2018

doc: Split build linux dependencies fad95e8da6

MarcoFalke force-pushed on Sep 19, 2018

MarcoFalke commented at 4:24 PM on September 19, 2018: member

Tested that the current instructions work on no fresh and minimal ubuntu/debian due to missing packages or not-found packages.

MarcoFalke force-pushed on Sep 19, 2018

MarcoFalke commented at 5:40 PM on September 19, 2018: member

Tested that depends compilation (and cross-compilation) works now with fad16ac14df1190de4c9fa9ee2e5ae89c421f0d3 on debian:9, xenial and bionic.

MarcoFalke added this to the milestone 0.17.1 on Sep 19, 2018

MarcoFalke removed this from the milestone 0.17.1 on Sep 19, 2018

in depends/README.md:51 in fad16ac14d outdated

  52 | +
  53 | +    sudo apt-get install make automake cmake curl g++-multilib libtool binutils-gold bsdmainutils pkg-config python3
  54 | +
  55 | +For linux ARM cross compilation:
  56 | +
  57 | +    sudo apt-get install g++-aarch64-linux-gnu binutils-aarch64-linux-gnu

arowser commented at 11:36 PM on September 19, 2018:

line 51 is for aarch64, I guess you should swap with line 55.

MarcoFalke force-pushed on Sep 20, 2018

in depends/README.md:39 in fa6e22eafe outdated

  38 | +#### For macOS cross compilation:
  39 |  
  40 |      sudo apt-get install curl librsvg2-bin libtiff-tools bsdmainutils cmake imagemagick libcap-dev libz-dev libbz2-dev python-setuptools
  41 |  
  42 | -For Win32/Win64 cross compilation:
  43 | +#### For Win32/Win64 cross compilation:

practicalswift commented at 8:17 AM on September 21, 2018:

2018-09-19 20:32:54 mdl(pr=14264): depends/README.md:39: MD026 Trailing punctuation in header

in depends/README.md:43 in fa6e22eafe outdated

  43 | +#### For Win32/Win64 cross compilation:
  44 |  
  45 |  - see [build-windows.md](../doc/build-windows.md#cross-compilation-for-ubuntu-and-windows-subsystem-for-linux)
  46 |  
  47 | -For linux (including i386, ARM) cross compilation:
  48 | +#### For linux (including i386, ARM) cross compilation:

practicalswift commented at 8:17 AM on September 21, 2018:

2018-09-19 20:32:54 mdl(pr=14264): depends/README.md:43: MD026 Trailing punctuation in header

in depends/README.md:35 in fa6e22eafe outdated

  29 | @@ -30,29 +30,38 @@ Common `host-platform-triplets` for cross compilation are:
  30 |  
  31 |  No other options are needed, the paths are automatically configured.
  32 |  
  33 | -Install the required dependencies: Ubuntu & Debian
  34 | ---------------------------------------------------
  35 | +### Install the required dependencies: Ubuntu & Debian
  36 |  
  37 | -For macOS cross compilation:
  38 | +#### For macOS cross compilation:

practicalswift commented at 8:17 AM on September 21, 2018:

2018-09-19 20:32:54 mdl(pr=14264): depends/README.md:47: MD026 Trailing punctuation in header

MarcoFalke force-pushed on Sep 21, 2018

DrahtBot commented at 6:44 AM on September 28, 2018: member

Coverage	Change (pull 14264)	Reference (master)
Lines	-0.0706 %	87.0361 %
Functions	+0.0000 %	84.1130 %
Branches	-0.0483 %	51.5451 %

MarcoFalke commented at 4:53 AM on October 3, 2018: member

@Sjors I've stolen this from you. Mind to take a look here if it looks right?

Sjors commented at 3:27 AM on October 4, 2018: member

It looks OK to me, but only way to know for sure is to try on a fresh machine, which I haven't gotten around to.

MarcoFalke commented at 3:58 AM on October 4, 2018: member

I am going to merge, since the current instructions are broken and I tried this on several fresh machines.

MarcoFalke merged this on Oct 4, 2018

MarcoFalke closed this on Oct 4, 2018

MarcoFalke referenced this in commit 69a29b5a8e on Oct 4, 2018

MarcoFalke deleted the branch on Oct 4, 2018

deadalnix referenced this in commit 3aaf582f6e on Mar 6, 2020

ftrader referenced this in commit 2043fee4b2 on Aug 17, 2020

MarcoFalke locked this on Sep 8, 2021