This is a snapshot of Indico's old Trac site. Any information contained herein is most probably outdated. Access our new GitHub site here.

Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#348 closed defect (fixed)

hard-coded OS-dependent path separator

Reported by: jherr Owned by: jherr
Priority: low Milestone: v0.97.0
Component: General Version: 0.97.0
Keywords: Cc:

Description

I found '%s/%s' used in several places instead of os.path.join()

Change History (3)

comment:1 Changed 6 years ago by jherr

  • Status changed from new to accepted

comment:2 Changed 6 years ago by jeremy.herr@…

  • Resolution set to fixed
  • Status changed from accepted to closed

commit 266df7e8c1a3c0ae383b97fd575f7938f7749d69
Author: Jeremy Herr <jeremy.herr@…>
Date: Thu Mar 25 12:49:28 2010 +0100

[FIX] Hard-coded path separators


  • I found several instances of %s/%s and changed them to os.path.join(), not including URLs of course.
  • I also found the method testInstallShouldProperlySetMaKaCConfigReferenceToIndicoConf() with hardcoded / and an abs path with /'s in it, but didn't change it.
  • fixes #348

comment:3 Changed 5 years ago by Jeremy Herr <jeremy.herr@…>

In [266df7e8c1a3c0ae383b97fd575f7938f7749d69]:

[FIX] Hard-coded path separators

  • I found several instances of %s/%s and changed them to os.path.join(), not including URLs of course.
  • I also found the method testInstallShouldProperlySetMaKaCConfigReferenceToIndicoConf() with hardcoded / and an abs path with /'s in it, but didn't change it.
  • fixes #348
Note: See TracTickets for help on using tickets.