Skip to content

Conversation

@tcinbis
Copy link

@tcinbis tcinbis commented Mar 21, 2019

Because the download URL for Geneious Prime slightly diverges from the
information found in the versions.txt an additional RegexReplace Processor is required.
With the modified download recipe both versions (20 and 2019) can be downloaded by
modifying the input variables.

As discussed in autopkg/autopkg#442 it makes more sense to add the processor for only this recipe, as not many other recipes currently need a RegexReplace processor.

Because the download url for Geneious Prime slightly diverges from the
information found in the versions.txt an additional RegexReplace Processor is required.
With the modified download recipe both version (20 and 2019) can be downloaded by
modifiying the input variables.
@tcinbis
Copy link
Author

tcinbis commented Apr 17, 2019

@CWhits , @n8felton do we need any further improvements or changes before we could merge this?

@tcinbis
Copy link
Author

tcinbis commented Jun 20, 2019

@CWhits , @n8felton giving this a little ping/push...

@chelming
Copy link

I'll test and merge this tomorrow.

Copy link

@chelming chelming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change

@tcinbis
Copy link
Author

tcinbis commented Jul 9, 2019

Bump @CWhits

Copy link

@chelming chelming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check the formatting on the regex replace processor. I don't think it will work as-is.

</dict>
<dict>
<key>Processor</key>
<string>RegexReplace</string>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this needs to be formatted like so:
<string>com.github.n8felton.shared/RegexReplace</string>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants