Skip to content

Add a recipe to build replan central#1663

Merged
jeanconn merged 3 commits intomasterfrom
replan_central
Mar 12, 2026
Merged

Add a recipe to build replan central#1663
jeanconn merged 3 commits intomasterfrom
replan_central

Conversation

@jeanconn
Copy link
Contributor

@jeanconn jeanconn commented Mar 5, 2026

Add a recipe to build replan central.

This works with the build in the "package" branch in sot/arc#96 .

Tested against ac4c84b using ska_builder.py on Linux only.

@jeanconn jeanconn requested a review from javierggt March 11, 2026 16:18
@jeanconn jeanconn marked this pull request as ready for review March 11, 2026 16:18
Copy link
Contributor

@javierggt javierggt left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me.

I created a package using the command documented in the skare3 wiki:

from skare3_tools.github import Repository

repo = Repository("sot/skare3")
repo.dispatch_event(
    event_type='build-package',
    client_payload={
        "package": "arc",
        "tag": "package",
        "arch": True,
        "upload": True,
        "skare3_branch": "replan_central",
    }
)

The resulting package fails to install on OSX because the package depends on packages that are only for linux (perl-extended-deps, perl-ska-convert). Was that intended?

I installed it on a test environment on HEAD and that worked. I then just checked that the entrypoints were there. No further functional check.

@jeanconn
Copy link
Contributor Author

Yeah, I figured it was safer to only let this install on linux, but if that causes more problems for management, I can cut the dependencies in the recipe. Either way.

@javierggt
Copy link
Contributor

It does not cause any problem for management

@jeanconn jeanconn merged commit eba6d2d into master Mar 12, 2026
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.

2 participants