import os.path

[Erdem ARTAN]

Bazı yerlerde şöyle birşey geçiyor:

if sys.platform == "win32":
        pisi = open(os.getcwd()+"\\packages\\"+pisiname,"rb")
    else:
        pisi = open(os.getcwd()+"/packages/"+pisiname,"rb")

Bunun yerine,

pisi = open(os.path.join(os.getcwd(), "packages" + pisiname, "rb")

kullansak daha güzel olmaz mı?



Erdem Artan

[Metehan Özbek]

"packages/" olması gerekiyor sanırım. Kodu kısaltıyorsa kullanalım.

[Erdem ARTAN]

evet sonunda / olması gerekiyor o koda göre.

kısaltıyor. win mac lin için ayrı iflerden kurtuluruz böylece.


Erdem Artan

2010/4/16 Metehan Özbek <met...@istanbul.com>

[Metehan Özbek]

Bana uyar...

[Erdem ARTAN]

düzenlemeleri yaptım, denedim, çalıştırdım ve commit ettim.

siz de bir deneyin isterseniz, gözümden birşey kaçtıysa sonradan başımız ağrımasın.



Erdem Artan

On Sat, Apr 17, 2010 at 00:20, Metehan Özbek <met...@istanbul.com> wrote:

Bana uyar...

[Metehan Özbek]

mesala "/package" kullanıldı ya bu Windows da sorun çıkartabilir. os.sep i kullanabilirsek ayraçların sorunuda ortadan kalkar.

downloaderda "/" koymamışın 189. satıra

liburl.py de de copy/paste yapmışın sanırım bazı kodlarıda silmişin :)

if os.path.isdir(os.getcwd()+"\\packages") == False:
    
os.mkdir("packages")

karşılaştırmalarını silmişsin.
http://code.google.com/p/pisidown/source/diff?spec=svn152&r=152&format=side&path=/trunk/src/liburl.py

[Erdem ARTAN]

/ kullanmaya gerek yok. os.path.join bu işi yapıyor.

liburlyi düzelttim.



Erdem Artan

2010/4/17 Metehan Özbek <met...@istanbul.com>