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

Installation of eprints 3.3.12 breaks IRStats2 plugin #48

Closed
dpavlin opened this issue Apr 17, 2014 · 5 comments
Closed

Installation of eprints 3.3.12 breaks IRStats2 plugin #48

dpavlin opened this issue Apr 17, 2014 · 5 comments

Comments

@dpavlin
Copy link

@dpavlin dpavlin commented Apr 17, 2014

After upgrade to 3.3.12 I get following error when trying to update stats (from cron when running /opt/eprints3/archives/ffzg/bin/stats/process_stats):

Error in configuration:
Can't call method "add_trigger" on unblessed reference at /usr/share/eprints3/archives/ffzg/bin/../cfg/cfg.d/z_irstats2.pl line 163

------------------------------------------------------------------
EPrints System Error inducing stack dump
 at /usr/share/eprints3/archives/ffzg/bin/stats/../../../../perl_lib/EPrints.pm line 146
        EPrints::abort('EPrints') called at /usr/share/eprints3/archives/ffzg/bin/stats/../../../../perl_lib/EPrints/Config.pm line 151
        EPrints::Config::load_system_config() called at /usr/share/eprints3/archives/ffzg/bin/stats/../../../../perl_lib/EPrints/Config.pm line 96
        EPrints::Config::init() called at /usr/share/eprints3/archives/ffzg/bin/stats/../../../../perl_lib/EPrints.pm line 706
        require EPrints.pm called at /opt/eprints3/archives/ffzg/bin/stats/process_stats line 12
        main::BEGIN() called at /usr/share/eprints3/archives/ffzg/bin/stats/../../../../perl_lib/EPrints.pm line 0
        eval {...} called at /usr/share/eprints3/archives/ffzg/bin/stats/../../../../perl_lib/EPrints.pm line 0

I tried upgrade to latest version from git, but it didn't help. Anything else I can try?

@sebastfr
Copy link
Contributor

@sebastfr sebastfr commented May 6, 2014

There's a potential fix here: eprints/eprints#232

@dpavlin
Copy link
Author

@dpavlin dpavlin commented May 6, 2014

I guess you are referring to eprints/eprints@c03adc8 but with it irstats dies on me trying to init session in

my $session = new EPrints::Session( 1, $repoid );

inside bin/stats/process_stats. I can't explain why, but if you have hints how to debug this more, I will take a stab at it.

@dpavlin
Copy link
Author

@dpavlin dpavlin commented May 26, 2014

However, without patching eprints, I can fix this problem with modification to bin/stats/process_stats. Instead of

use FindBin;
use lib "$FindBin::Bin/../../../../perl_lib";

I used just

use lib '/opt/eprints3/perl_lib';

because eprints had regex which expect eprints3.perl_lib without parent directory ../../../ paths in it. I could submit patch which cleans up lib path if needed so that IRStats will work with any eprints version.

@drtjmb
Copy link
Member

@drtjmb drtjmb commented May 27, 2014

Can you try this patch which (should) fix the root problem:

eprints/eprints@c03adc8

On 26 May 2014 12:25, Dobrica Pavlinušić notifications@github.com wrote:

However, without patching eprints, I can fix this problem with
modification to bin/stats/process_stats. Instead of

use FindBin;
use lib "$FindBin::Bin/../../../../perl_lib";

I used just

use lib '/opt/eprints3/perl_lib';

because eprints had regex which expect eprints3.perl_lib without parent
directory ../../../ paths in it. I could submit patch which cleans up lib
path if needed so that IRStats will work with any eprints version.


Reply to this email directly or view it on GitHubhttps://github.com//issues/48#issuecomment-44180579
.[image: Web Bug from
https://github.com/notifications/beacon/2921909__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcxNjcyMjc0MCwiZGF0YSI6eyJpZCI6MzAyMjk1OTd9fQ==--5181bddf996ffa49931975a1b61b3d56f67cc74a.gif]

@sebastfr
Copy link
Contributor

@sebastfr sebastfr commented Jul 3, 2014

Fixed by eprints patch (above).

@sebastfr sebastfr closed this Jul 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants