Bug 252836 - textproc/rubygem-mdl: Add missing sysutils/rubygem-chef-utils dependency
Summary: textproc/rubygem-mdl: Add missing sysutils/rubygem-chef-utils dependency
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: Ryan Steinmetz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-01-19 20:49 UTC by Daniel Ebdrup Jensen
Modified: 2021-03-30 21:07 UTC (History)
4 users (show)

See Also:

Attachments
A svn patch for Makefile (633 bytes, patch)
2021-01-19 20:49 UTC, Daniel Ebdrup Jensen 		no flags Details | Diff
Traceback showing missing dependency (1.43 KB, text/plain)
2021-01-19 20:50 UTC, Daniel Ebdrup Jensen 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Ebdrup Jensen freebsd_committer 2021-01-19 20:49:08 UTC 
Created attachment 221748 [details]
A svn patch for Makefile

When mdl is started after being installed from packages, it complains about the fact that rubygem-chef-utils is missing.

See attachment for full log.

I've also attempted to make a patch, which I believe should work.
Comment 1 Daniel Ebdrup Jensen freebsd_committer 2021-01-19 20:50:01 UTC 
Created attachment 221749 [details]
Traceback showing missing dependency
Comment 2 Daniel Ebdrup Jensen freebsd_committer 2021-02-19 12:51:11 UTC 
Maintainer apparently didn't get tagged automatically Bugzilla.
Comment 3 Zane C. Bowers-Hadley 2021-02-19 16:09:10 UTC 
Awesome! Thanks. Looks good.
Comment 4 Mikael Urankar freebsd_committer 2021-02-19 16:50:08 UTC 
approved by: mikael
Comment 5 commit-hook freebsd_committer 2021-02-19 17:24:32 UTC 
A commit references this bug:

Author: zi
Date: Fri Feb 19 17:24:18 UTC 2021
New revision: 566084
URL: https://svnweb.freebsd.org/changeset/ports/566084

Log:
  - Add missing dependency
  - Bump PORTREVISION

  PR:		252836
  Submitted by:	debdrup@
  Approved by:	maintainer

Changes:
  head/textproc/rubygem-mdl/Makefile
Comment 6 commit-hook freebsd_committer 2021-03-03 18:21:24 UTC 
A commit references this bug:

Author: sunpoet
Date: Wed Mar  3 18:18:13 UTC 2021
New revision: 567245
URL: https://svnweb.freebsd.org/changeset/ports/567245

Log:
  Fix r558727

  - Add missing dependency
  - Bump PORTREVISION for dependency change

  It also fixes textproc/rubygem-mdl runtime issue.

  PR:		252836
  Approved by:	portmgr (blanket)

Changes:
  head/devel/rubygem-mixlib-shellout/Makefile
Comment 7 Po-Chuan Hsieh freebsd_committer 2021-03-03 18:36:26 UTC 
chef-utils is an indirect dependency via mixlib-shellout [1].
r567245 should fix the problem.

In order to keep the dependency correct [2], please revert r566084.
Thanks!

[1] https://rubygems.org/gems/mixlib-shellout/versions/3.2.2
[2] https://rubygems.org/gems/mdl/versions/0.11.0
Comment 8 commit-hook freebsd_committer 2021-03-30 19:27:26 UTC 
A commit references this bug:

Author: sunpoet
Date: Tue Mar 30 19:27:06 UTC 2021
New revision: 569592
URL: https://svnweb.freebsd.org/changeset/ports/569592

Log:
  Revert r566084, fixed in r567245

  - Bump PORTREVISION for dependency change

  chef-utils is an indirect dependency via mixlib-shellout [1].
  I've fixed the problem in r567245.

  In order to keep the dependency correct [2], I reverted r566084
  which added rubygem-chef-utils to the wrong place.

  [1] https://rubygems.org/gems/mixlib-shellout/versions/3.2.2
  [2] https://rubygems.org/gems/mdl/versions/0.11.0

  PR:		252836
  Reference:	https://bugs.FreeBSD.org/bugzilla/show_bug.cgi?id=252836#c7
  Approved by:	portmgr (blanket)

Changes:
  head/textproc/rubygem-mdl/Makefile