[MacPorts] #40516: Two new variants + text editor modeline
#40516: Two new variants + text editor modeline -------------------------+-------------------------------- Reporter: jpolmsted@… | Owner: macports-tickets@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Keywords: | Port: R -------------------------+-------------------------------- I added two new variants to the current R Portfile. check: runs the "check-all" tests for the R build during the Test Macports phase. tests: installs testing code and output for subsequent testing of the installation during the destroot Macports phase. Also, per the documentation, I added a modeline up top. -- Ticket URL: <https://trac.macports.org/ticket/40516> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Changes (by macsforever2000@…): * owner: macports-tickets@… => kjell.konis@… * keywords: => haspatch * version: 2.2.0 => Comment: Thanks. In the future, please Cc the port maintainers ({{{port info --maintainers R}}}). -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by larryv@…): Comments: - Setting `test.run yes` does not actually run any tests; it just //enables// running them via `port test R`. So the `test.*` options should be at the root level of the Portfile, and there should not be a variant. (Otherwise you’d have to run `port test R +check`.) - Would it be detrimental to just run `make install-tests` always, instead of inside a variant? -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by jpolmsted@…): I'm confused about the opposition. This is behavior intended by me and seems completely consistent with the test phase. Am I missing something? Under this setup, checking is optional (as it is with a manual install). From an R user's perspective, I don't think there is a downside. It just copies files to the installation prefix that already exist and can be used. -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by larryv@…): Replying to [comment:3 jpolmsted@…]:
I'm confused about the opposition. This is the behavior intended by me and seems completely consistent with the test phase. Am I missing something? Under this setup, checking is optional (as it is with a manual install) and it happens after the build and before the install.
This is not true; the test phase is //not// run during a port installation, even if the Portfile sets `test.run yes`. Users would already have to explicitly run the tests with `port test R`, so they should not have to select a variant on top of that.
From an R user's perspective, I don't think there is a downside. It just copies files to the installation prefix that already exist and can be used.
Hm. If installing the tests isn’t terribly inconvenient (build time, disk space, etc.), then they should probably always be installed. -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by jpolmsted@…): Replying to [comment:4 larryv@…]:
Replying to [comment:3 jpolmsted@…]:
I'm confused about the opposition. This is the behavior intended by me and seems completely consistent with the test phase. Am I missing something? Under this setup, checking is optional (as it is with a manual install) and it happens after the build and before the install.
This is not true; the test phase is //not// run during a port installation, even if the Portfile sets `test.run yes`. Users would already have to explicitly run the tests with `port test R`, so they should not have to select a variant on top of that.
Thanks, that makes sense. I'll update the patch tomorrow to be consistent with `port test R` usage.
From an R user's perspective, I don't think there is a downside. It just copies files to the installation prefix that already exist and can be used.
Hm. If installing the tests isn’t terribly inconvenient (build time, disk space, etc.), then they should probably always be installed.
I don't disagree, but I'll err on the side of the R developers here. They didn't (and still don't) choose to include them. But the port maintainer can easily change it if wanted. -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by jpolmsted@…): I've updated the patch. For the current version: * Test phase-specific changes are root level. * Installation of tests for later use still occurs only by requesting the "tests" variant. * Text editor modeline added to the Portfile. -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by kjell.konis@…): Maintainer here. Looks good to me. Thanks for the patch. The next update to R is scheduled for the 25th so I'll make these changes with the version bump next week. -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by kjell.konis@…): Maintainer here. The R version bump in #40602 (1) adds the text editor modeline (2) enables testing (i.e., sets test.run to yes). I set test.target to check rather than check-all because check-all requires that additional CRAN packages be installed in order complete cleanly (3) adds a test variant that installs the tests for later use -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:9> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: closed Priority: Normal | Milestone: Component: ports | Version: Resolution: fixed | Keywords: haspatch Port: R | --------------------------+--------------------------- Changes (by larryv@…): * status: new => closed * resolution: => fixed Comment: r111649 -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:10> MacPorts <http://www.macports.org/> Ports system for OS X
#40516: Two new variants + text editor modeline --------------------------+--------------------------- Reporter: jpolmsted@… | Owner: kjell.konis@… Type: enhancement | Status: closed Priority: Normal | Milestone: Component: ports | Version: Resolution: fixed | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by egall@…): Replying to [comment:4 larryv@…]:
Replying to [comment:3 jpolmsted@…]:
I'm confused about the opposition. This is the behavior intended by me and seems completely consistent with the test phase. Am I missing something? Under this setup, checking is optional (as it is with a manual install) and it happens after the build and before the install.
This is not true; the test phase is //not// run during a port installation, even if the Portfile sets `test.run yes`. Users would already have to explicitly run the tests with `port test R`, so they should not have to select a variant on top of that.
I've always found this counterintuitive. There should be some sort of option to automatically run the test phase during a normal port installation... -- Ticket URL: <https://trac.macports.org/ticket/40516#comment:11> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts