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

irstats2 plugin conflict (add_dataset_field errors) #41

Closed
cziaarm opened this issue Mar 7, 2014 · 5 comments
Closed

irstats2 plugin conflict (add_dataset_field errors) #41

cziaarm opened this issue Mar 7, 2014 · 5 comments

Comments

@cziaarm
Copy link

@cziaarm cziaarm commented Mar 7, 2014

Hi Seb,

Just installed irstats2 from epm v 2.0.0.5 (on EP 3.3.12)

When trying to run process_stats I get the error:

Can't call method "add_dataset_field" on unblessed reference at /data/www/lshtmdr-test/eprints3/archives/lshtmdrtest/bin/../cfg/cfg.d/request_copy.pl line 71.

EPrints (on reload or restart) does not have a problem with this piece of config and it works well, it seems to be only bin/process_stats that has an issue?

If I deal with that line of code it will move along and complain about the next call to add_dataset_field. Is it possibly some problem with the way process_stats is evaluating the cfg.d stuff out of context? Or a problem with all the other plugins?

Cheers,

Rory

@sebastfr
Copy link
Contributor

@sebastfr sebastfr commented Mar 7, 2014

I've seen this in rare instances.... Try commenting out all the stuff related to FindBin in bin/process_stats and if needed hard-code the path to perl_lib (e.g. via -I/opt/eprints3/.... or use lib "/opt/e....").

FindBin should just work... but in this case it just doesn't!

Btw I'm planning on releasing a new minor version of stats2 soon-ish.

@sebastfr
Copy link
Contributor

@sebastfr sebastfr commented Mar 7, 2014

So in other words it's not $c->add.... that's causing the problems but rather the way perl_lib (and the conf etc) is loaded via FindBin in this particular instance. If you manage to have this one fixed, I'll be happy :-)

@cziaarm
Copy link
Author

@cziaarm cziaarm commented Mar 10, 2014

Oooh That's a stinker! I Had an attempt at a fix, but in the end had to hard-code the path in process_stats as you suggested. They are playing happily now, which is the main thing. Thanks!

@sebastfr
Copy link
Contributor

@sebastfr sebastfr commented Mar 13, 2014

Rory, on which OS did you install this? I'm trying to find reasons why this is sometimes working/failing.

@cziaarm
Copy link
Author

@cziaarm cziaarm commented Mar 13, 2014

Hi Seb,

That would be RHEL 5.7

Rory

On 13/03/2014 16:08, Sebastien Francois wrote:

Rory, on which OS did you install this? I'm trying to find reasons why
this is sometimes working/failing.


Reply to this email directly or view it on GitHub
#41 (comment).

Rory McNicholl
Lead developer, Research Repositories Team
Academic Research Technologies
University of London Computer Centre
Senate House
Malet Street
London
WC1E 7HU

t: +44 (0)20 7863 1344
e: rory.mcnicholl@london.ac.uk
w: http://www.ulcc.ac.uk/
b: http://dablog.ulcc.ac.uk/

To ensure you receive the full benefits of the repositories service
please remember to cc repositories@ulcc.ac.uk

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
2 participants