Skip to content

ddclient: Update to v4.0.0 - #9977

Open
androidnisse wants to merge 1 commit into
getsolus:mainfrom
androidnisse:ddclient
Open

ddclient: Update to v4.0.0#9977
androidnisse wants to merge 1 commit into
getsolus:mainfrom
androidnisse:ddclient

Conversation

@androidnisse

Copy link
Copy Markdown
Contributor

Summary

  • Release notes can be found here.
  • ddclient now looks for ddclient.conf in ${sysconfdir}/ddclient by default instead of ${sysconfdir}.

Test Plan

  • Checked the new build works, that files are in correct places and right permissions. Not updated for dynamic domain as I have no account for that.

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged
  • I agree to license this contribution and all my previous contributions under the licensing terms in LICENSE.md and have the power and authority to grant those licenses.

Comment on lines +18 to +19
./autogen
%configure

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

%autogen

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This package is "stupid" the file is not named autogen.sh only autogen. That is why I used that command.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🤦

Two potential options:

  1. Rename it to autogen.sh
  2. Add %CONFOPTS% to ./autogen so our configure options are applied. Assuming they actually accept standard autotools options...

Comment thread packages/d/ddclient/package.yml Outdated
**Summary**
- Release notes can be found [here](https://github.com/ddclient/ddclient/releases/tag/v4.0.0).
- ddclient now looks for ddclient.conf in ${sysconfdir}/ddclient by default instead of ${sysconfdir}.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

2 participants