Bug 245337 - [NEW PORT] devel/sentry-cli: Command line utility to work with Sentry
Summary: [NEW PORT] devel/sentry-cli: Command line utility to work with Sentry
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Tobias Kortkamp
URL: https://github.com/getsentry/sentry-cli
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-04 00:12 UTC by Lewis Cook
Modified: 2020-04-04 09:34 UTC (History)
0 users

See Also:


Attachments
sentry-cli-1.52.1.diff (49.40 KB, patch)
2020-04-04 00:12 UTC, Lewis Cook
no flags Details | Diff
sentry-cli-1.52.1.log (294.52 KB, text/plain)
2020-04-04 00:12 UTC, Lewis Cook
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lewis Cook freebsd_committer freebsd_triage 2020-04-04 00:12:28 UTC
Created attachment 213039 [details]
sentry-cli-1.52.1.diff

Sentry command line client for some generic tasks.
It can connect to the Sentry API and manage some
data for your projects. It's primarily used for
managing debug information files for iOS, Android
as well as release and source maps management for
other platforms.

WWW: https://docs.sentry.io/cli/

Testing:
   * No errors reported with portlint, built in Poudriere and
     application works as expected (logs attached).
Comment 1 Lewis Cook freebsd_committer freebsd_triage 2020-04-04 00:12:57 UTC
Created attachment 213040 [details]
sentry-cli-1.52.1.log

* Poudriere build log.
Comment 2 Tobias Kortkamp freebsd_committer freebsd_triage 2020-04-04 07:27:50 UTC
This has the same problem as net/findomain had: it builds a bundled copy
of OpenSSL (openssl-src) instead of using the system one.  It is easy
to take care of that but it is a worrying trend in the Rust ecosystem
that this is the default configuration. :-(
Comment 3 commit-hook freebsd_committer freebsd_triage 2020-04-04 07:32:40 UTC
A commit references this bug:

Author: tobik
Date: Sat Apr  4 07:29:55 UTC 2020
New revision: 530619
URL: https://svnweb.freebsd.org/changeset/ports/530619

Log:
  New port: devel/sentry-cli

  Sentry command line client for some generic tasks.
  It can connect to the Sentry API and manage some
  data for your projects. It's primarily used for
  managing debug information files for iOS, Android
  as well as release and source maps management for
  other platforms.

  WWW: https://docs.sentry.io/cli/

  PR:		245337
  Submitted by:	Lewis Cook <vulcan@wired.sh>

Changes:
  head/devel/Makefile
  head/devel/sentry-cli/
  head/devel/sentry-cli/Makefile
  head/devel/sentry-cli/distinfo
  head/devel/sentry-cli/files/
  head/devel/sentry-cli/files/patch-Cargo.toml
  head/devel/sentry-cli/pkg-descr
Comment 4 commit-hook freebsd_committer freebsd_triage 2020-04-04 07:51:44 UTC
A commit references this bug:

Author: tobik
Date: Sat Apr  4 07:40:38 UTC 2020
New revision: 530623
URL: https://svnweb.freebsd.org/changeset/ports/530623

Log:
  Mk/Uses/cargo.mk: Add a warning about the openssl-src crate

  PR:		245337

Changes:
  head/Mk/Uses/cargo.mk
Comment 5 Lewis Cook freebsd_committer freebsd_triage 2020-04-04 09:25:12 UTC
(In reply to Tobias Kortkamp from comment #2)

Initially I was reluctant on even posting this submission, but thanks for the heads up. Wishing there was a more trivial way of handling this other than explicitly including a patch in each port that pulls in the openssl-sys crate.

For what it's worth, would it be an idea adding an extra step in the patch target of Cargo.mk - addressing this issue? I imagine it's easier said than done.
Comment 6 Lewis Cook freebsd_committer freebsd_triage 2020-04-04 09:34:43 UTC
(In reply to Lewis Cook from comment #5)

Oops, s/openssl-sys/openssl-src/ *.

Overlooked the commit from earlier in Cargo.mk with the message. Looks like the right approach for now. :)