Ftp site commands in xbfilezilla lib

[clonexb]

i was told recently that the ftp site commands had been commented out in teh xbfilezilla source code, so i download the whole xbmc source and have been looking at the code and it appears that the code has not been commented out but there are some wierdo callback funtions which i think you are supposed to setup your own version of a callback for each site command. i dont think any have even been setup and i have managed to get site execute and site reboot to work by simply just adding my own code to handle it but without setting up any callbacks.

does anybody know any more about this and what is the correct way is to do it and also does anyone know if full site command support is already there and i am just not enabling it in the correct way ?

clonexb.

[pike]

afaik, this was started but never finished.
you're welcome to fix it up and commit patches :o

[clonexb]

actually i may just do that, would there be any objection to me not using the wierdo callback thingy ? , its not like you will ever get a response anyway from most site commands anyway as its usually a command which kill the current ftp connection such as reboot, shutdown and launch xbe

[jmarshall]

anything is better than nothing.

none of the main devs have time to look into this sort of stuff at the moment. if you could do up a patch for the stuff you can get working (i think the best way would be to use the cutil::execbuiltin() function if possible) that'd be great. we can look into the callback stuff at a later stage.

cheers,
jonathan

[clonexb]

what would you feel about me adding a site layout command in addition to the other site commands ?

i am the author of clonexb and xdvdfs_tools which allow you to create backups that maintain the same physical file layout as the original disc which makes for faster loading backups.

anyway this backup method is becoming more and more popular, see http://layouts.xbox-scene.com/ for more info.

basically the site layout command would produce a .lyt file on the hd, this file contains a list of all files on the dvd, which layer they are on, actual sector positions, file lengths and some other useful info needed to produce a backup with the same layout. what do you think ?

clonexb.

ps: i would have to include a new library into the source for this to happen.

[jmarshall]

if you think it'd be a useful addition, then go for it.

how big is the library?

[clonexb]

ok cool, i have never compiled it as a lib before, i have some sorting out to do first but it should be very small as its not doing an awful lot really.