Re: [Scid-users] build scid on Windows

[Cristian S. <cri...@gm...>]

It's not that big of a problem as I thought. I just reproduced it on Linux
too. It's enough to have spaces in path for scid to trigger the bug.

I've identified the problem in InitImg procedure in this loop:

    foreach {piecetype} [split [glob -directory $dname *] " "] {
        puts "piecetype $piecetype"
        if {[file isdirectory $piecetype] == 1} {
            lappend boardStyles [file tail $piecetype]
        }
    }

"split" breaks on the wrong assumption and doesn't create the list you
would expect.
I'll look at it tomorrow since it's late for me now.

Cristian


On Thu, May 30, 2013 at 10:41 PM, Mike Curtis <mic...@gm...>wrote:

> Okay, thanks.
>
> My seldom-used windows platform is building scid with Cygwin, so I have no
> chance to reproduce the bug you found.
>
> Well, I checked it anyway, just to be sure.
>
> If I had that bug, I would keep adding instrumentation, until the point of
> failure became obvious.
>
> For example
>
>  diff --git a/tcl/bitmaps.tcl b/tcl/bitmaps.tcl
> index 2ce27c4..232ad78 100644
> --- a/tcl/bitmaps.tcl
> +++ b/tcl/bitmaps.tcl
> @@ -78,6 +78,10 @@ proc setPieceFont {font} {
>                 set fname [file join $::scidImgDir pieces $font]
>                 set fpng [file join $fname $font\_$size.png]
>                 set fgif [file join $fname $font\_$size.gif]
> +puts "font = $font"
> +puts "fname = $fname"
> +puts "fpng = $fpng"
> +puts "fgif = $fgif"
>                 if {[catch {image create photo tmpPieces -file "$fpng"}]} {
>
>                         image create photo tmpPieces -file "$fgif"
>                 }
> @@ -102,10 +106,12 @@ proc InitImg {} {
>                         lappend boardStyles [file tail $piecetype]
>                 }
>         }
> +puts "1) boardStyle = $boardStyle"
>         # Ensure the board style is valid:
>         if {[lsearch -exact $boardStyles $boardStyle] == -1} {
>                 set boardStyle [lindex $boardStyles 0]
>         }
> +puts "2) boardStyle = $boardStyle"
>         setPieceFont $boardStyle
>
>         #Load all img/buttons/_filename_.png
>
>
>
> On Thu, May 30, 2013 at 1:31 PM, Cristian Stoica <cri...@gm...>wrote:
>
>> Only the folders are created (config, log and data) but they are empty. I
>> tried wit an options.dat from 4.4 but nothing changed.
>>
>> Cristian
>>
>>
>> On Thu, May 30, 2013 at 2:23 PM, Mike Curtis <mic...@gm...>wrote:
>>
>>> What happens if you rename options.dat to options.BAK and try again?
>>>
>>> On Thu, May 30, 2013 at 5:57 AM, Cristian Stoica <cri...@gm...>wrote:
>>>
>>>> I want to check why it wants a _25.gif file. This is just a suffix to
>>>> some of the piece files that are one level deeper than /img/pieces.
>>>> Something is not right.
>>>>
>>>> Anyway, config dir being present or not does not change the result and
>>>> the img/pieces has the image dirs.
>>>>
>>>> Cristian
>>>>
>>>>
>>>>
>>>>
>>>> On Thu, May 30, 2013 at 10:33 AM, Fulvio <fb...@li...> wrote:
>>>>
>>>>> Cristian Stoica wrote:
>>>>>
>>>>>> Hi Fulvio,
>>>>>>
>>>>>> I don't know what I did, but for a brief period of time I could run
>>>>>> the build without any problems. After reboot the problem came back. Here is
>>>>>> the trace. If I put a _25.gif file there, it complains about a missing
>>>>>> _30.gif.
>>>>>>
>>>>>>  Delete the Release/config directory and be sure that inside
>>>>> img/pieces there are the Alpha, Bauhaus, etc.. directories
>>>>> Bye,
>>>>> Fulvio
>>>>>
>>>>
>>>>
>>>>
>>>> ------------------------------------------------------------------------------
>>>> Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET
>>>> Get 100% visibility into your production application - at no cost.
>>>> Code-level diagnostics for performance bottlenecks with <2% overhead
>>>> Download for free and get started troubleshooting in minutes.
>>>> http://p.sf.net/sfu/appdyn_d2d_ap1
>>>> _______________________________________________
>>>> Scid-users mailing list
>>>> Sci...@li...
>>>> https://lists.sourceforge.net/lists/listinfo/scid-users
>>>>
>>>>
>>>
>>
>