Bug 252593 - sysutils/datadog-integrations: Update to 7.24.0
Summary: sysutils/datadog-integrations: Update to 7.24.0
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Kai Knoblich
URL: https://github.com/DataDog/integratio...
Depends on:
Reported: 2021-01-11 19:24 UTC by uros
Modified: 2021-03-06 12:23 UTC (History)
2 users (show)

See Also:

patch for new version (5.15 KB, patch)
2021-01-11 19:24 UTC, uros
uros: maintainer-approval+
Details | Diff
Fixed formating (6.08 KB, patch)
2021-01-14 10:14 UTC, uros
no flags Details | Diff
format changes (7.24 KB, patch)
2021-01-14 12:36 UTC, uros
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description uros 2021-01-11 19:24:35 UTC
Created attachment 221476 [details]
patch for new version

Update datadog-integrations to the latest 7.24.0
Comment 1 Fernando Apesteguía freebsd_committer 2021-01-13 12:17:14 UTC
^Triage: If there is a changelog or release notes URL available for this version, please add it to the URL field.

^Triage: Maintainer-feedback flag (+) not required unless requested (?) first.

Q/A: Some small improvements to the Makefile:

# USES block

# Standard bsd.port.mk variables

# Users and groups block



Comment 2 uros 2021-01-13 12:49:10 UTC
There is no changelog avialable from their site. And with port I mainly just increased version number.

Should I do the change in Makefile and update patch file? Sorry if I ask such questions but I'm ferly new to this processes.
Comment 3 Fernando Apesteguía freebsd_committer 2021-01-13 12:57:38 UTC
(In reply to uros from comment #2)
No need to apologize!

If there is no changelog, it is OK. Thanks for checking that.

Yes, please, it would be great if you could make the changes. Have a look at: https://www.freebsd.org/doc/en/books/porters-handbook/book.html#porting-order

You can use ports-mgmt/portfmt and ports-mgmt/portlint to double check your ports :-)

Comment 4 uros 2021-01-14 10:14:18 UTC
Created attachment 221557 [details]
Fixed formating

Please check with this patch. Should be all good now.
Comment 5 Fernando Apesteguía freebsd_committer 2021-01-14 11:45:02 UTC
(In reply to uros from comment #4)
Almost done.

# Standard bsd.port.mk variables

Comment 6 uros 2021-01-14 12:36:16 UTC
Created attachment 221558 [details]
format changes
Comment 7 Fernando Apesteguía freebsd_committer 2021-01-14 14:09:25 UTC
(In reply to uros from comment #6)
Comment 8 uros 2021-01-19 21:01:34 UTC
Just checking if there is anything else I need to fix?
Comment 9 Fernando Apesteguía freebsd_committer 2021-01-20 10:36:55 UTC
(In reply to uros from comment #8)
LGTM, good jon!

Wait for a committer to pick it up
Comment 10 commit-hook freebsd_committer 2021-03-06 12:06:18 UTC
A commit references this bug:

Author: kai
Date: Sat Mar  6 12:05:30 UTC 2021
New revision: 567483
URL: https://svnweb.freebsd.org/changeset/ports/567483

  sysutils/datadog-integrations: Update to 7.24.0

  * Also sort some variables according the PHB and fix indents.



  PR:		252593
  Submitted by:	Uro? Gruber (maintainer)
  Reviewed by:	fernape

Comment 11 Kai Knoblich freebsd_committer 2021-03-06 12:23:21 UTC
Committed, thank you both for the patch and the review!

I did some additional fixes to the pkg-plist to avoid build failures if built with all options set.