mailRe: Independence of the new lib.system module.


Others Months | Index by Date | Thread Index
>>   [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Header


Content

Posted by Troels Emtekær Linnet on May 12, 2016 - 12:42:
Thank you!

Best
Troels

2016-05-12 9:21 GMT+02:00 Edward d'Auvergne <edward@xxxxxxxxxxxxx>:

On 20 April 2016 at 11:41, Edward d'Auvergne <edward@xxxxxxxxxxxxx> wrote:
Hi Troels,

In running the test suite, I noticed the following test failing:

$ ./relax --verification-tests Library.test_library_independence

The problem is the lib.system module, in that you cannot import the
status module.  Instead you should call
status.observers.system_cwd_path.notify() after any calls to
lib.system.cd(), i.e. outside of the relax library, rather than in
this function itself.

Hi Troels,

In preparation for a new relax release, I'll fix this issue myself.  I
am simply shifting lib.system to pipe_control.system.  The
pipe_control package is mainly for processing arguments, providing
feedback, calling other code to do the work, and updating the relax
state.  But as the system.cd and system.pwd user functions are almost
entirely composed of argument processing, feedback, and updating the
state, I think that it can just all go there.  I'll also update the
copyright notices with your new name :)

Regards,

Edward

_______________________________________________
relax (http://www.nmr-relax.com)

This is the relax-devel mailing list
relax-devel@xxxxxxx

To unsubscribe from this list, get a password
reminder, or change your subscription options,
visit the list information page at
https://mail.gna.org/listinfo/relax-devel



Related Messages


Powered by MHonArc, Updated Fri May 13 04:20:13 2016