Skip to content

Commit

Permalink
Merge branch 'master' into add-notice-files-to-source-units
Browse files Browse the repository at this point in the history
  • Loading branch information
spatten authored Sep 10, 2024
2 parents aeb5b56 + 10e2abc commit 8e6c0d0
Show file tree
Hide file tree
Showing 13 changed files with 263 additions and 107 deletions.
2 changes: 2 additions & 0 deletions .cargo/config.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[target.aarch64-unknown-linux-musl]
rustflags = ["-C", "link-arg=-lgcc"]
201 changes: 109 additions & 92 deletions .github/workflows/build-all.yml

Large diffs are not rendered by default.

31 changes: 31 additions & 0 deletions .github/workflows/scripts/build-arm.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
#!/usr/bin/env sh

set -e
set -x

if [ $# -lt 2 ] ; then
echo "Usage: ./build-arm.sh <RUNNER OS> <PROJECT FILE> [FEATURES]"
exit 1
fi

RUNNER_OS=$1
PROJECT_FILE=$2
FEATURES=$3

# Install rust tooling
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --profile minimal
. "/home/runner/.cargo/env"

rustc -V
cargo -V

# Build Rust binaries
cargo build ${FEATURES:+--features $FEATURES} --release

cargo test --release

# Validate that diagnose runs
cargo run --release --bin diagnose -- walk --trace-spans none --trace-level info

# Now do Haskell things
./.github/workflows/scripts/build.sh "$RUNNER_OS" "$PROJECT_FILE"
36 changes: 36 additions & 0 deletions .github/workflows/scripts/build.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
#!/usr/bin/env sh

set -e
set -x

if [ $# -lt 2 ] ; then
echo "Usage: ./build.sh <RUNNER OS> <PROJECT FILE>"
exit 1
fi

RUNNER_OS=$1
PROJECT_FILE=$2

make build-test-data

# Does this need to run in the container?
# It normally runs outside.
git config --global --add safe.directory "$GITHUB_WORKSPACE"

# With dist-newstyle caches:
# Cabal mainly knows to recompile based on changes to files.
# Tagging in git doesn't reliably change a file in a fixed location that cabal/GHC can track to indicate that there's a new tag.
# For our release process, we merge to master, which builds (and may store a dist-newstyle cache), then push a release tag.
# During the tag build, cabal/GHC may not realize that they have to rebuild the Version.hs file because the tag is invisible to it.
# This line adds a comment to our version source file to prompt cabal/GHC to rebuild Version.hs.
echo "{- $GITHUB_RUN_ID -}" >> src/App/Version.hs
cabal update
cabal build --project-file="$PROJECT_FILE" all
cabal test --project-file="$PROJECT_FILE" unit-tests

# Todo: Bring this back.
# Test cabal-install.
# This check ensures that QuickImport can use spectrometer as a library.
# if [ "$RUNNER_OS" = 'Linux' ] ; then
# cabal install --overwrite-policy=always --project-file="$PROJECT_FILE" --ghc-options="-Wwarn"
# fi
34 changes: 34 additions & 0 deletions .github/workflows/scripts/compute_cache_key.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
#!/usr/bin/env sh

set -e
set -x

if [ $# -lt 2 ] ; then
echo "Usage: ./compute_cache_key <RUNNER OS> <PROJECT FILE>"
exit 1
fi

RUNNER_OS=$1
PROJECT_FILE=$2

cabal --project-file="$PROJECT_FILE" update
cabal --project-file="$PROJECT_FILE" build --dry-run
jq '."install-plan"[]."id"' < dist-newstyle/cache/plan.json | sort > /tmp/cabal-cache-key
echo "Install plan:"
cat /tmp/cabal-cache-key

if [ "$RUNNER_OS" = "macOS" ]; then
PLAN_SUM=$(shasum -a256 /tmp/cabal-cache-key)
else
PLAN_SUM=$(sha256sum /tmp/cabal-cache-key)
fi

CABAL_CACHE_KEY="$(echo "$PLAN_SUM" | awk '{print $1}')"
export CABAL_CACHE_KEY
echo "Cabal cache key: $CABAL_CACHE_KEY"
echo "cabal-cache-key=$CABAL_CACHE_KEY" >> "$GITHUB_OUTPUT"

echo "Home dotfiles in container: $(ls -al ~/.*)"

# Cleanup. Restoring this cache seems to fail if the directory already exists.
rm -rf dist-newstyle
2 changes: 1 addition & 1 deletion Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,7 @@ resolver = "2"
[profile.release]
lto = "fat"
strip = true

# There is additional configuration in .cargo/config.toml.
# It is used for building on aarch64-unknown-linux-musl.
# More context on the split here: https:/rust-lang/cargo/issues/12738
4 changes: 4 additions & 0 deletions Changelog.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# FOSSA CLI Changelog

## 3.9.32

- Platform Support: Add a binary for ARM64 Linux environments. ([#1465](https:/fossas/fossa-cli/pull/1465))

## 3.9.31

- Resolve an issue parsing toml configuration files. ([#1459](https:/fossas/fossa-cli/pull/1459))
Expand Down
2 changes: 1 addition & 1 deletion extlib/millhone/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ tikv-jemallocator = { version = "0.5.4", optional = true }
clap = { version = "4.3.21", features = ["derive", "env", "cargo"] }
stable-eyre = "0.2.2"
srclib = { version = "*", git = "https:/fossas/foundation-libs" }
snippets = { version = "0.1.3", git = "https:/fossas/foundation-libs", features = ["lang-all"] }
snippets = { version = "0.1.3", tag = "v0.1.3", git = "https:/fossas/lib-snippets", features = ["lang-all"] }
traceconf = { git = "https:/fossas/foundation-libs", version = "1.1.0" }
serde = { version = "1.0.183", features = ["derive"] }
thiserror = "1.0.46"
Expand Down
13 changes: 12 additions & 1 deletion install-latest.sh
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ is_supported_platform() {
darwin/amd64) found=0 ;;
darwin/arm64) found=0 ;;
linux/amd64) found=0 ;;
linux/arm64) found=0 ;;
esac
return $found
}
Expand Down Expand Up @@ -119,7 +120,16 @@ adjust_arch() {
# adjust archive name based on ARCH
true
}

arch_version_check() {
# TODO: Make this version correct before merging.
if [ "${OS}/${ARCH}" = "linux/arm64" ] && version_less_than "${VERSION}" '3.9.32'
then
echo "There is no linux/arm64 binary for version $VERSION."
echo "Please select a version that is at least version 3.9.32"
exit 1
fi
true
}
cat /dev/null <<EOF
------------------------------------------------------------------------
https:/client9/shlib - portable posix shell functions
Expand Down Expand Up @@ -482,6 +492,7 @@ adjust_os

adjust_arch

arch_version_check

NAME=$(get_binary_name)
TARBALL=${NAME}.${FORMAT}
Expand Down
14 changes: 10 additions & 4 deletions src/Data/FileEmbed/Extra.hs
Original file line number Diff line number Diff line change
@@ -1,16 +1,17 @@
module Data.FileEmbed.Extra (
embedFile',
embedFileIfExists,
) where
)
where

import Control.Exception (try)
import Control.Monad (when)
import Data.FileEmbed (embedFile)
import Data.FileEmbed (embedFile, makeRelativeToProject)
import Data.Foldable (traverse_)
import Data.Maybe (isJust)
import Language.Haskell.TH (reportWarning, runIO)
import Language.Haskell.TH.Syntax (Exp (LitE), Lit (..), Q)
import Path (parseRelFile)
import Path (parseAbsFile)
import Path.IO (doesFileExist)
import System.Directory (getCurrentDirectory, listDirectory)
import System.Environment (lookupEnv)
Expand All @@ -19,7 +20,12 @@ import System.FilePath (isRelative, splitPath, takeDirectory)
embedFileIfExists :: FilePath -> Q Exp
embedFileIfExists inputPath = do
skipEmbedEnvVar <- runIO $ lookupEnv "FOSSA_SKIP_EMBED_FILE_IN_HLS"
case (skipEmbedEnvVar, parseRelFile inputPath) of
-- During some commands (like cabal install) the working directory may not
-- be the same as the root of this project.
-- makeRelativeToProject takes a relative path and roots it in the highest directory
-- a .cabal file is found and returns an absolute path.
inputPath' <- makeRelativeToProject inputPath
case (skipEmbedEnvVar, parseAbsFile inputPath') of
(Just _, _) -> do
pure (LitE $ StringL "")
(_, Just path) -> do
Expand Down
10 changes: 6 additions & 4 deletions test/App/Fossa/PathDependencySpec.hs
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import Control.Algebra (Has)
import Control.Effect.FossaApiClient (FossaApiClientF (..), PackageRevision (PackageRevision))
import Data.Flag (toFlag)
import Data.Set qualified as Set
import Data.String.Conversion (toText)
import Data.Text (Text)
import DepTypes (
DepEnvironment (EnvTesting),
Expand Down Expand Up @@ -41,10 +42,11 @@ absPathOfSpec = describe "absPathOfSpec" $ do
mkPathSpec cwd "../" isAbsDir
mkPathSpec cwd "./" isAbsDir

mkPathSpec cwd "../fossa-cli" isAbsDir
mkPathSpec cwd "../fossa-cli/" isAbsDir
mkPathSpec cwd "../fossa-cli/test" isAbsDir
mkPathSpec cwd "../fossa-cli/test/" isAbsDir
let currentPathRel = "../" <> (toText . toFilePath . dirname $ cwd)
mkPathSpec cwd currentPathRel isAbsDir
mkPathSpec cwd (currentPathRel <> "/") isAbsDir
mkPathSpec cwd (currentPathRel <> "/test") isAbsDir
mkPathSpec cwd (currentPathRel <> "/test/") isAbsDir

mkPathSpec cwd "./test" isAbsDir
mkPathSpec cwd "./test/" isAbsDir
Expand Down
17 changes: 13 additions & 4 deletions vendor_download.sh
Original file line number Diff line number Diff line change
Expand Up @@ -51,11 +51,20 @@ case "$(uname -s)" in
;;

Linux)
ASSET_POSTFIX="linux"
THEMIS_ASSET_POSTFIX="linux-amd64"
LERNIE_ASSET_POSTFIX="x86_64-linux"
;;
case "$(uname -m)" in
aarch64)
ASSET_POSTFIX="linux"
THEMIS_ASSET_POSTFIX="linux-arm64"
LERNIE_ASSET_POSTFIX="aarch64-linux"
;;

*)
ASSET_POSTFIX="linux"
THEMIS_ASSET_POSTFIX="linux-amd64"
LERNIE_ASSET_POSTFIX="x86_64-linux"
;;
esac
;;
*)
echo "Warn: Assuming $(uname -s) is Windows"
ASSET_POSTFIX="windows.exe"
Expand Down

0 comments on commit 8e6c0d0

Please sign in to comment.