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.

Sign up for GitHub

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

Jump to bottom

don't destroy (possibly un-uploaded) .tar.gz #308

Open
mralusw opened this issue May 27, 2021 · 1 comment · May be fixed by #309
Open

don't destroy (possibly un-uploaded) .tar.gz #308

mralusw opened this issue May 27, 2021 · 1 comment · May be fixed by #309

Comments

@mralusw
Copy link

mralusw commented May 27, 2021

Minilla has an annoying habit of destroying the generated .tar.gz distribution, even if

  • it was a dry-run or FAKE_RELEASE run
  • CPAN::Uploader failed (e.g. due to bad password)

To get back the .tar.gz and manually upload, one has to

  • remove the date header from the not-uploaded release
  • run minil dist
    which is not so bad once you figure it out, but it does look like user-hostile behavior.

Perhaps default (or offer an option at least) to save the .tar.gz. I've currently added a shell option to the "Upload to CPAN?" prompt in UploadToCPAN.pm (my fork), but it's really just a work-around (another is to open a new terminal etc).
@mralusw mralusw linked a pull request May 27, 2021 that will close this issue
Open
@mralusw
Copy link
Author

mralusw commented May 27, 2021

A related problem is that the created .tar.gz archives contain actual UNIX usernames. GNU tar has --numeric-owner, but I don't think Archive::Tar can support that. So, opening a shell just before uploading to CPAN allows one to re-pack the archive and strip usernames.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

UploadToCPAN.pm: shell option at prompt, use env var for password kstr0k/Minilla
1 participant
@mralusw