Modified build.sh for x86_64-unknown-linux-gnu - resolve Compiling warnings

[rekab72]

I am new to Rust but I modified build.sh for x86_64-unknown-linux-gnu (Ubuntu 64 on Intel).

Compiling imessage-exporter ends with warnings. Any insights appreciated. I am investigating by Googling.

build.sh modified

# Create output directory
mkdir -p output

# Get latest tags
git pull

# Get version number from tag name
export VERSION=$(git describe --tags $(git rev-list --tags --max-count=1))

if [ -n "$VERSION" ]; then
    # Confirm version number with user before build
    read -p "Build with version $VERSION? C-c to cancel, any key to continue " -n 1 -r
    echo

    # Update version number in Cargo.toml for build
    # macOS sed requires the weird empty string param
    # Otherwise it returns `invalid command code C`
    sed -i '' "s/version = \"0.0.0\"/version = \"$VERSION\"/g" imessage-database/Cargo.toml
    sed -i '' "s/version = \"0.0.0\"/version = \"$VERSION\"/g" imessage-exporter/Cargo.toml
    sed -i '' s/'{path = "..\/imessage-database"}'/\"$VERSION\"/g imessage-exporter/Cargo.toml

    if [ -n "$PUBLISH" ]; then
        echo 'Publishing database library...'
        cargo publish -p imessage-database --allow-dirty

        echo 'Publishing exporter binary...'
        cargo publish -p imessage-exporter --allow-dirty
    else
        echo 'PUBLISH env var not set!'
    fi

    # Build for 64-bit Intel Linux
    cargo build --target x86_64-unknown-linux-gnu --release
    cp target/x86_64-unknown-linux-gnu/release/imessage-exporter output/imessage-exporter-x86_64-unknown-linux-gnu
    cd target/x86_64-unknown-linux-gnu/release/
    tar -czf ../../../output/imessage-exporter-x86_64-unknown-linux-gnu.tar.gz imessage-exporter
    cd ../../..

    # Put the version number back
    sed -i '' "s/version = \"$VERSION\"/version = \"0.0.0\"/g" imessage-database/Cargo.toml
    sed -i '' "s/version = \"$VERSION\"/version = \"0.0.0\"/g" imessage-exporter/Cargo.toml
    sed -i '' s/\"$VERSION\"/'{path = "..\/imessage-database"}'/g imessage-exporter/Cargo.toml

    unset VERSION
else
    echo 'No version tag set!'
fi

Output_cargo_build_Ubuntu_X86_64

$ cargo build

   Compiling libc v0.2.126
   Compiling cfg-if v1.0.0
   Compiling autocfg v1.1.0
   Compiling version_check v0.9.4
   Compiling once_cell v1.15.0
   Compiling cc v1.0.76
   Compiling vcpkg v0.2.15
   Compiling pkg-config v0.3.24
   Compiling serde v1.0.145
   Compiling bitflags v1.3.2
   Compiling num_threads v0.1.6
   Compiling ahash v0.7.6
   Compiling safemem v0.3.3
   Compiling portable-atomic v0.3.19
   Compiling num-traits v0.2.14
   Compiling indexmap v1.9.1
   Compiling num-integer v0.1.44
   Compiling ppv-lite86 v0.2.16
   Compiling itoa v1.0.4
   Compiling regex-syntax v0.6.27
   Compiling getrandom v0.2.3
   Compiling time v0.1.43
   Compiling terminal_size v0.1.17
   Compiling rand_core v0.6.3
   Compiling time v0.3.15
   Compiling hashbrown v0.12.3
   Compiling rand_chacha v0.3.1
   Compiling line-wrap v0.1.1
   Compiling xml-rs v0.8.4
   Compiling libsqlite3-sys v0.25.1
   Compiling hashlink v0.8.1
   Compiling regex v1.6.0
   Compiling fallible-iterator v0.2.0
   Compiling unicode-width v0.1.10
   Compiling iana-time-zone v0.1.53
   Compiling base64 v0.13.1
   Compiling smallvec v1.7.0
   Compiling fallible-streaming-iterator v0.1.9
   Compiling os_str_bytes v6.0.0
   Compiling console v0.15.0
   Compiling chrono v0.4.23
   Compiling rand v0.8.5
   Compiling clap_lex v0.2.4
   Compiling atty v0.2.14
   Compiling termcolor v1.1.2
   Compiling number_prefix v0.4.0
   Compiling strsim v0.10.0
   Compiling textwrap v0.16.0
   Compiling indicatif v0.17.3
   Compiling uuid v1.2.2
   Compiling clap v3.2.23
   Compiling plist v1.3.1
   Compiling filetime v0.2.19
   Compiling rusqlite v0.28.0
   Compiling imessage-database v0.0.0 (/home/owner/imessage-exporter-1.2.0/imessage-database)
   Compiling imessage-exporter v0.0.0 (/home/owner/imessage-exporter-1.2.0/imessage-exporter)
warning: unused `Result` that must be used
   --> imessage-exporter/src/exporters/html.rs:102:17
    |
102 |                 msg.gen_text(&self.config.db);
    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
    = note: `#[warn(unused_must_use)]` on by default

warning: unused `Result` that must be used
   --> imessage-exporter/src/exporters/html.rs:359:25
    |
359 |                         reply.gen_text(&self.config.db);
    |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled

warning: unused `Result` that must be used
  --> imessage-exporter/src/exporters/txt.rs:88:17
   |
88 |                 msg.gen_text(&self.config.db);
   |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: this `Result` may be an `Err` variant, which should be handled

warning: unused `Result` that must be used
   --> imessage-exporter/src/exporters/txt.rs:243:25
    |
243 |                         reply.gen_text(&self.config.db);
    |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled

warning: `imessage-exporter` (bin "imessage-exporter") generated 4 warnings
    Finished dev [unoptimized + debuginfo] target(s) in 10.75s

[ReagentX]

1. Those are warnings, not errors, you can ignore them. There are no errors here; the build was successful. Here is some detail on those warnings.

2. You shouldn't need to use build.sh. If you just want to build yourself you can use cargo build.

[rekab72]

Thanks. I modified build.sh thinking it was the "build" that $cargo build referenced. I am very new to Rust so I didn't know what drove the target as you suggest in another answer that was possible. My misunderstanding of how $cargo build gets the target.

[ReagentX]

build.sh is how I create the builds I upload to the releases page. Perhaps I should rename that to avoid any confusion, because all users need to do is use cargo as noted in the docs.

[rekab72]

Thank you for insight/direction back to docs. It works!!!!!!! Now to give it a try.

cargo install imessage-exporter is the best way to install the app for normal use.

Compiling imessage-database v1.2.0
   Compiling imessage-exporter v1.2.0
    Finished release [optimized] target(s) in 42.39s
  Installing /home/owner/.cargo/bin/imessage-exporter
   Installed package `imessage-exporter v1.2.0` (executable `imessage-exporter`)