[RTEMS Project] #2375: tftpDriver free's current_directory

[RTEMS trac]

#2375: tftpDriver free's current_directory
-------------------------+---------------------
 Reporter:  mdavidsaver  |       Owner:
     Type:  defect       |      Status:  new
 Priority:  normal       |   Milestone:  4.10.3
Component:  networking   |     Version:  4.10
 Severity:  normal       |  Resolution:
 Keywords:               |
-------------------------+---------------------

Comment (by mdavidsaver):

 Replying to [comment:5 gedare]:
 > Replying to [comment:4 mdavidsaver]:
 > ...
 > The appropriateness of this fix to their users. (Which is to say, maybe
 someone else who is impacted by the bug or the fix might be available to
 review the code?)

 At present TFTP support is a compile time conditional choice between TFTP
 or NFS, with the default being NFS.  IMO the age of this bug is an
 indication of how many sites change this as it breaks ~100% of the time.
 I'd like to make this a runtime choice.

 My motivation for this actually relates to the second part of your reply.
 For the past year or so we've been working to increase the unit-test
 coverage in EPICS Base.  Part of this was setting up continuous
 integration (Jenkins) to automatically build and run these tests.

 This works great for Linux, Windows, and similar OSs, but not for RTEMS or
 vxWorks.  We have tests for these targets as well, but it's a pain to run
 them, so it isn't often done.  This led to some last minute pain in recent
 release cycles.

 I've been working to automatically run these tests with RTEMS in QEMU,
 which could be done by a CI server.  It's working great against RTEMS
 4.9.6 and helped me find a bug in some other code I'm working on.

 https://github.com/mdavidsaver/epics-base/tree/multiboot

 (Don't get too excited about the branch name, it should have been
 something like "bspcmdline")

 ...
 > For now, it might be best to put the test into the network-demos
 repository. The existing Makefile should just about work there, and
 someone can do the 'waf' part easily enough.

 Fine by me.

--
Ticket URL: <http://devel.rtems.org/ticket/2375#comment:8>
RTEMS Project <http://www.rtems.org/>
RTEMS Project