#1783 linker requires .lk file, docs say .lnk

closed-fixed
linker (61)
5
2013-05-25
2011-04-26
No

Somewhere between 6382 (Apr 7th) and 6468 (Apr 26th), the linker was changed to only accept the filename extension ".lk". I'm not sure why this is, but it deviates from the documentation (sdas/dos/asmlnk.txt) which states [ in -f file mode ] "the commands are input from the specified file (extension must be .lnk).".

It looks like this was introduced by Revision 6445, where lklex.c was modified to change "lnk" to "lk".

In addition to this deviating from the docs, it also deviates from the usage output which still states "file[.lnk]" as the source for the "-f" argument.

Whilst I don't really mind which extension is used, the documentation and implementation should match.

I'm not sure how to map this back to a "bug" number which was fixed, so I can't see the rationale behind the change.

SDCC versions involved:
Works with .lnk as documented: SDCC : z80 3.0.2 #6382 (Apr 7 2011) (Solaris i386)
Requires .lk extension: SDCC : z80 3.0.2 #6468 (Apr 26 2011) (Solaris i386)

Error produced with the later version:

$ bin/sdldz80 -nf project.lnk
?ASlink-Error-<cannot open> : "project.lk"
*** Error code 3

Discussion

  • Maarten Brock

    Maarten Brock - 2011-04-28

    Hello Brian,

    I changed it to .lk but still have to adapt the documentation. The reason is that Windows, even with the option to show extensions, hides this extension because it thinks this is a link to another file/folder. It is also my intention to use whatever extension is privided and default to .lk when none is given.

    Maarten

     
  • Brian Ruthven

    Brian Ruthven - 2011-04-28

    Thanks for the explanation. At least I know it's not a glitch, and I can modify my project accordingly.

     
  • Maarten Brock

    Maarten Brock - 2011-05-03
    • milestone: --> fixed
    • assigned_to: nobody --> maartenbrock
    • status: open --> closed-fixed
     
  • Maarten Brock

    Maarten Brock - 2011-05-03

    I fixed the documentation (asmlnk.txt, asxhtm.html, sdld -h) and also implemented the use of the provided extension in revision #6476.

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks