Bug 241227 - ports-mgmt/portmaster: When installing/reinstalling/upgrading devel/cmake portmaster always reinstall textproc/py-sphinx even if it is up to date
Summary: ports-mgmt/portmaster: When installing/reinstalling/upgrading devel/cmake por...
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Stefan Esser
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-10-13 15:18 UTC by Yasuhiro KIMURA
Modified: 2019-10-23 04:48 UTC (History)
6 users (show)

See Also:


Attachments
patch proposal for devel/cmake (474 bytes, patch)
2019-10-14 07:36 UTC, Tatsuki Makino
no flags Details | Diff
and... (2.55 KB, text/plain)
2019-10-14 07:45 UTC, Tatsuki Makino
no flags Details
patch proposal for portmaster (621 bytes, patch)
2019-10-14 23:57 UTC, Tatsuki Makino
no flags Details | Diff
patch proposal for devel/cmake (612 bytes, patch)
2019-10-23 04:48 UTC, Tatsuki Makino
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yasuhiro KIMURA 2019-10-13 15:18:35 UTC
When installing/reinstalling/upgrading devel/cmake portmaster always reinstall textproc/py-sphinx even if it is up to date.

----------------------------------------------------------------------
root@rolling-vm-freebsd1[2001]# portmaster cmake-3.15.4                                                                    ~

===>>> Currently installed version: cmake-3.15.4
===>>> Port directory: /net/freebsd/ports/head/devel/cmake

===>>> Launching 'make checksum' for devel/cmake in background
===>>> Gathering dependency list for devel/cmake from ports
===>>> Launching child to install textproc/py-sphinx

===>>> cmake-3.15.4 >> textproc/py-sphinx (1/1)

===>>> Currently installed version: py37-sphinx-1.6.5_2,1
===>>> Port directory: /net/freebsd/ports/head/textproc/py-sphinx

===>>> Launching 'make checksum' for textproc/py-sphinx in background
===>>> Gathering dependency list for textproc/py-sphinx from ports
===>>> Initial dependency check complete for textproc/py-sphinx

===>>> Continuing initial dependency check for devel/cmake
===>>> Initial dependency check complete for devel/cmake


===>>> cmake-3.15.4 >> (1)

===>>> The following actions will be taken if you choose to proceed:
        Re-install cmake-3.15.4
        Re-install py37-sphinx-1.6.5_2,1

===>>> Proceed? y/n [y] n


===>>> If you would like to upgrade or install some, but not
       all of the above try adding '-i' to the command line.
root@rolling-vm-freebsd1[2002]#
----------------------------------------------------------------------
Comment 1 Raphael Kubo da Costa freebsd_committer 2019-10-13 15:27:07 UTC
This looks like an issue in portmaster, tentatively assigning to the port maintainer.
Comment 2 Tatsuki Makino 2019-10-14 07:36:13 UTC
Created attachment 208291 [details]
patch proposal for devel/cmake

(In reply to Raphael Kubo da Costa from comment #1)

Perhaps this is a well-reported bug. I also thought it was a bug in portmaster.
However, I think this is a bug on the devel/cmake side.
Comment 3 Tatsuki Makino 2019-10-14 07:45:46 UTC
Created attachment 208292 [details]
and...

These are also likely to happen... astro/geographiclib! Really? orz
Comment 4 Yasuhiro KIMURA 2019-10-14 14:21:07 UTC
(In reply to Tatsuki Makino from comment #2)

But if I try "cd /usr/ports/devel/cmake && make" then build completes without reinstall textproc/py-sphinx. So please tell me why this is bug of devel/cmake rather than portmaster.
Comment 5 Tatsuki Makino 2019-10-14 23:57:50 UTC
Created attachment 208319 [details]
patch proposal for portmaster

(In reply to Yasuhiro KIMURA from comment #4)

There is no problem with the canonical method, but it is unkind to automation tools.

If we fix the portmaster side, probably here.
Comment 6 Tobias C. Berner freebsd_committer 2019-10-17 19:17:48 UTC
To give some context:

This line in the cmake-doc Makefile was changed in r460715 to be flavored, and backed out again in r485766 -- the consesus at the time was, that just depending on the default py-versioned sphinx binary is the correct thing. See PR 227573 .
Comment 7 Yasuhiro KIMURA 2019-10-22 00:51:05 UTC
(In reply to Tobias C. Berner from comment #6)

Thank you for information. Then the problem is bug of portmaster.
Comment 8 Tatsuki Makino 2019-10-23 04:48:41 UTC
Created attachment 208518 [details]
patch proposal for devel/cmake

(In reply to Tobias C. Berner from comment #6)

Can bug 227573 be solved this patch?

Is it cmake side? :)