> Firstly, thanks for offering this package.

My pleasure, and I appreciate the helpful feedback provided!


> for historical reasons, the lines lines within the .cygport file defining 
> NAME, VERSION and RELEASE

Thank you, I will adjust the Cygport file accordingly.


> "RESTRICT=diff" is probably appropriate here

Yes, this is just what is needed; thank you.

A related item that I observed here is that—at least in the case of the code 
here—the "--no-dereference" command-line argument to diff perhaps might 
sometimes be useful?  There are symbolic links in the codebase, and if there 
are changes to source during the compilation process and that source is also 
referenced by symbolic link(s) elsewhere, the same file diffs being included 
multiple times via different symlink paths.


> I think CYGWIN_NOWINPATH isn't having any effect in your workflow

On some further digging, this appears to be exactly the case.  The script is 
detecting that Ruby is installed in the runner, but that detected version is 
not the Cygwin build but instead one found at 
"/c/hostedtoolcache/windows/Ruby/3.3.8/x64/bin/ruby".  As a workaround, I then 
installed the Cygwin build of Ruby on the GitHub runner, and then "cygport 
package" step completes without issue.


Thank you,
Matthew

Reply via email to