[pypy-dev] os.major? os.minor?

Benjamin Peterson benjamin at python.org
Fri Dec 10 20:50:36 CET 2010


2010/12/10 Dan Stromberg <drsalists at gmail.com>:
> On Thu, Dec 9, 2010 at 10:33 AM, Armin Rigo <arigo at tunes.org> wrote:
>> Hi,
>>
>> Sorry for the series of mails.  I just wanted to add that after
>> looking more at the code, it's fine if you implement major() and
>> minor() the same way as the other posix functions (see e.g. r79941
>> adding os.fchdir()), calling the major() and minor() "functions" in C.
>>  If you add tests in the same places I did in r79941, then it should
>> work even if the functions are actually macros.  One place where it
>> would not work is in translated code running on the llinterp; the
>> other place might be the JIT trying to get the address of the
>> "function".  (If that's the case, we should fix that a bit more
>> generally.)
>>
>>
>> A bientôt,
>>
>> Armin.
>
> I attempted to add os.major and os.minor using fchdir as an example,
> but...  I have to admit: I really don't know what to make of this
> error.  Comments?

You probably failed to add an ll implementation in
pypy/rpython/lltypesystem/module/ll_os.py


-- 
Regards,
Benjamin



More information about the Pypy-dev mailing list