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

mailx to s-nail transition #679

Open
fabbione opened this issue Jun 15, 2024 · 4 comments · Fixed by #684
Open

mailx to s-nail transition #679

fabbione opened this issue Jun 15, 2024 · 4 comments · Fixed by #684
Assignees
Labels
bug low low priority
Milestone

Comments

@fabbione
Copy link
Member

mailx has been deprecated in f39 in favor of s-nail.

The current external repo dependencies still carry mailx from f37.

For rhel-8/centos-8-stream we can just leave it as-is as both are being slowly replaced by 9.

For rhel-9/almalinux-9 we need to transition to a more recent version of all the external dependencies. That includes:

  1. add s-nail to the external repo (f40 based)
  2. change anvil spec file to Conflics mailx and Requires s-nail for >= 9
  3. change anvil code to use s-nail

It is unknown if s-nail is command line compatible with mailx, but it does provide the same functionalities.

Not targeting any specific anvil release for now as updating the external repo has to be coordinate and tested.

@fabbione
Copy link
Member Author

fabbione commented Jun 15, 2024

anvil]$ grep mailx * -r
Anvil/Tools.pm:                         mailx                           =>      "/usr/bin/mailx",
Anvil/Tools.pm:* C<mailx>
Anvil/Tools/Email.pm:# mailx.
Anvil/Tools/Email.pm:Provides all methods used to manage the local C<< postfix >> server and handle and dispatch email via C<< mailx >>
Anvil/Tools/Email.pm:           # Call mailx to read it in
Anvil/Tools/Email.pm:                   my $shell_call = $anvil->data->{path}{exe}{mailx}." -t < ".$file_name;
anvil.spec.in:Requires:       mailx
tools/striker-manage-install-target:                    "mailx.x86_64",

@digimer
Copy link
Member

digimer commented Jun 15, 2024

Thanks, I'll make the changes in a new PR shortly.

@fabbione
Copy link
Member Author

From a packaging perspective, I created a mailx transitional package that Requires: s-nail. This avoids issues with building the archive and updating anvil.spec, but you still need to look into the runtime. There is no compatibility symlink or anything.

digimer pushed a commit that referenced this issue Jul 16, 2024
* This resolves issue #679

Signed-off-by: Madison Kelly <[email protected]>
@ylei-tsubame ylei-tsubame reopened this Jul 22, 2024
@ylei-tsubame
Copy link
Collaborator

Adding Conflict: mailx for >= 9 breaks updates. When installing the new package with mailx already installed would cause a conflict error.

Maybe the solution is to use Obsoletes: mailx instead according to the guidelines. We can technically consider this mailx to s-nail transition as a rename as s-nail includes the same functions as mailx?

@fabbione fabbione added this to the Parking lot milestone Aug 28, 2024
@digimer digimer added enhancement bug low low priority and removed enhancement labels Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug low low priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants