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

Remove the PEAR/Archive_Tar requirement #25

Closed
jonasraoni opened this issue May 14, 2022 · 9 comments
Closed

Remove the PEAR/Archive_Tar requirement #25

jonasraoni opened this issue May 14, 2022 · 9 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@jonasraoni
Copy link
Collaborator

jonasraoni commented May 14, 2022

Some users have been complaining in the forum about a missing requirement "Your system must have a tar executable".
Given the package whikloj/bagittools already includes the PEAR package by itself composer, we don't need to check this requirement and the message might be removed to avoid confusions.

PRs

@jonasraoni jonasraoni added the enhancement New feature or request label May 14, 2022
@jonasraoni jonasraoni added this to the 2.0.4-3 milestone May 14, 2022
@jonasraoni jonasraoni self-assigned this May 14, 2022
@jonasraoni jonasraoni moved this to Backlog in Plugins and Themes May 14, 2022
jonasraoni added a commit to jonasraoni/pln that referenced this issue Jun 15, 2022
@jonasraoni jonasraoni moved this from Backlog to Under Development in Plugins and Themes Jun 16, 2022
@jonasraoni jonasraoni moved this from Backlog to Under Development in Metadata and Distribution Jun 16, 2022
jonasraoni added a commit to jonasraoni/pln that referenced this issue Jun 17, 2022
@jonasraoni
Copy link
Collaborator Author

@asmecher Here I've just removed the TAR requirement (we're not using it directly and the bagittools already includes such dependency to feed itself).

PRs

@asmecher
Copy link
Member

Looks good, I've merged both PRs -- but in retrospect I should've asked first. What does the PKP|PN staging server do with the tarInstalled status in the ping? If the requirement will no longer be sent to the staging server, are there any checks that will start generating errors/warnings?

@jonasraoni
Copy link
Collaborator Author

@asmecher I've checked this before removing, and the staging server isn't using most of the data provided by the ping.
Perhaps it was added as a way to spy/debug if the installation has everything that's needed without asking to the user 🤔

Repository owner moved this from Under Development to Done in Plugins and Themes Jun 21, 2022
Repository owner moved this from Under Development to Done in Metadata and Distribution Jun 21, 2022
@eddoff
Copy link

eddoff commented Nov 22, 2022

I read the thread https://forum.pkp.sfu.ca/t/ojs3-3-0-6-pkp-pn-plugin-warning-your-system-must-have-a-tar-executable/70273/7 because we have the same problem. However, I cannot find how to install this update, and I don't have access to Composer. To download the zip om the main branch from Git unpack that on the server did not work. Is there soon a new release that could be downloaded?

/eddoff

@jonasraoni
Copy link
Collaborator Author

If you don't have composer on your server, an alternative would be to generate the tar.gz with the dependencies on your machine.

@eddoff
Copy link

eddoff commented Nov 28, 2022

Do you have a link to a guide how to do that?

@jonasraoni
Copy link
Collaborator Author

There's some documentation, but it doesn't cover your case. The process is simple, but better to ask for help on the forum, here we're just handling triaged issues.

@eddoff
Copy link

eddoff commented Aug 10, 2023

Isn't it time for a new release soon? It will then be much easier for us to upgrade via the plugin gallery instead.

@jonasraoni
Copy link
Collaborator Author

Hi @eddoff,

We had issues within the Preservation Network, so I decided to delay the new release, as it would increase the number of received deposits and make things worse. Now that everything's ok, I'm just waiting for a review here: #28

Then a new release will happen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

3 participants