Issue1717492

classification
Title: os.path.ismount incompletely implemented
Type: Severity: normal
Components: None Versions:
Milestone:
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: amak, john.jython, mboersma, nriley, pekka.klarck, pjenvey
Priority: normal Keywords:

Created on 2007-05-11.22:35:17 by pekka.klarck, last changed 2008-09-13.22:41:17 by nriley.

Messages
msg1590 (view) Author: Pekka Klärck (pekka.klarck) Date: 2007-05-11.22:35:17
Currently this method always returns false.
msg1591 (view) Author: Alan Kennedy (amak) Date: 2007-05-19.12:50:17
I don't see any simple way to implement this on jython; java does not have not support for detecting mounted filesystems. At least that I can find in google or browsing the java 1.4 and 1.5 documentation.

Perhaps the submitter has some pointers on how this can be done in java? 

If it can't be done in java, it can't be done in jython.
msg1592 (view) Author: Pekka Klärck (pekka.klarck) Date: 2007-05-19.13:40:04
Unfortunately I don't have any idea how to implement this in Java/Jython. I submitted this because I noticed following at the top of javapath.py when I was writing patches to abspath and realpath bugs.

# Incompletely implemented:
# islink -- How?
# ismount -- How?
# splitdrive -- How?
# normcase -- How?

There already was a bug report for normcase [1] but I submitted also islink [2] and splitdrive [3]. I also created patches for all other than this one and Charlie has already committed most of them.

[1] http://jython.org/bugs/1648449
[2] http://jython.org/bugs/1717491
[3] http://jython.org/bugs/1717498

If there's no way to get this implemented I'd say simply documenting this is a good enough resolution. Not having ismount in Jython is not that big a problem especially because it doesn't work in Windows either.
msg3050 (view) Author: Philip Jenvey (pjenvey) Date: 2008-02-24.04:29:39
there's no way I can see of implementing this on posix. we now use the 
platform dependent path module (ntpath on Windows) as of r4171; ntpath 
actually includes a pure python version of ismount (it's easy to do on 
windows).

we might want to remove this from ntpath though, because people using it 
on windows might expect it to work on posix (since it exists in both 
places on CPython)
msg3051 (view) Author: Philip Jenvey (pjenvey) Date: 2008-02-24.20:09:35
ntpath.ismount was disabled in r4174
msg3092 (view) Author: John Sirbu (john.jython) Date: 2008-03-17.21:08:02
This functionality will be set aside; Currently this nice to have will
have to be placed on back burner as it don't meet target goals...
msg3530 (view) Author: Matt Boersma (mboersma) Date: 2008-09-13.19:50:28
ismount is implemented in jython 2.5a2 and later.
msg3547 (view) Author: Nicholas Riley (nriley) Date: 2008-09-13.22:41:16
Guess this is fixed then.
History
Date User Action Args
2008-09-13 22:41:17nrileysetstatus: open -> closed
nosy: + nriley
resolution: fixed
messages: + msg3547
2008-09-13 19:50:28mboersmasetnosy: + mboersma
messages: + msg3530
2008-03-17 21:08:02john.jythonsetnosy: + john.jython
messages: + msg3092
2008-02-24 20:09:36pjenveysetmessages: + msg3051
2008-02-24 04:29:40pjenveysetnosy: + pjenvey
messages: + msg3050
2007-05-11 22:35:17pekka.klarckcreate