Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop remnants of rake package:apple support #9367

Merged
merged 1 commit into from
May 29, 2024

Conversation

joshcooper
Copy link
Contributor

Shipping the nightly puppet gem is currently failing with:

bundle exec rake pl:ship_nightly_gem FOSS_ONLY=true ANSWER_OVERRIDE=false
rake aborted!
Could not locate a Rake task named 'cfpropertylist' to add as a dependency of 'package:apple'

This was broken in commit 5e4b124 as the packaging repo can no longer load the cfpropertylist rake task from the puppet repo.

The package:apple rake task was used to install a version of CFPropertyList to speed up plist reads/writes. However, that task has been broken since at least commit d988fe7 when the file_mapping.yaml file was deleted:

bundle exec rake package:apple
rake aborted!
Could not load Apple file mappings from 'ext/osx/file_mapping.yaml'

Since the package:apple rake task hasn't worked since 2021, delete the cfproperty rake task and the "pre_tasks" entry that registers the cfpropertylist task with the packaging gem.

@joshcooper joshcooper changed the title Drop remenants of rake package:apple support Drop remnants of rake package:apple support May 28, 2024
@joshcooper joshcooper marked this pull request as ready for review May 28, 2024 22:28
@joshcooper joshcooper requested a review from a team as a code owner May 28, 2024 22:28
@mhashizume mhashizume added the maintenance Maintenance chores are excluded from changelogs label May 28, 2024
@joshcooper
Copy link
Contributor Author

Need to rebase on #9368

Shipping the nightly puppet gem is currently failing with:

    bundle exec rake pl:ship_nightly_gem FOSS_ONLY=true ANSWER_OVERRIDE=false
    rake aborted!
    Could not locate a Rake task named 'cfpropertylist' to add as a dependency of 'package:apple'

This was broken in commit 5e4b124 as the packaging repo can no longer load
the cfpropertylist rake task from the puppet repo.

The `package:apple` rake task was used to install a version of CFPropertyList
to speed up plist reads/writes. However, that task has been broken since at
least commit d988fe7 when the file_mapping.yaml
file was deleted:

    bundle exec rake package:apple
    rake aborted!
    Could not load Apple file mappings from 'ext/osx/file_mapping.yaml'

Since the `package:apple` rake task hasn't worked since 2021, delete the
cfproperty rake task and the "pre_tasks" entry that registers the cfpropertylist
task with the packaging gem.
@joshcooper joshcooper merged commit 2d96e71 into puppetlabs:main May 29, 2024
9 checks passed
@joshcooper joshcooper deleted the drop_cfproperty_task branch May 29, 2024 16:51
@joshcooper joshcooper added the backport 7.x Generate a backport PR to 7.x label May 29, 2024
Copy link

Backport failed for 7.x, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git fetch origin 7.x
git worktree add -d .worktree/backport-9367-to-7.x origin/7.x
cd .worktree/backport-9367-to-7.x
git checkout -b backport-9367-to-7.x
ancref=$(git merge-base 77c3d6616625887466011a0e82a2f95ff60578b3 c8f1670bf8efd387d40565a50404d560885063eb)
git cherry-pick -x $ancref..c8f1670bf8efd387d40565a50404d560885063eb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 7.x Generate a backport PR to 7.x maintenance Maintenance chores are excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants