Skip to content

Add customized directories issues/21#37

Open
NonaSuomy wants to merge 71 commits into
FREEWING-JP:mainfrom
NonaSuomy:pr-36
Open

Add customized directories issues/21#37
NonaSuomy wants to merge 71 commits into
FREEWING-JP:mainfrom
NonaSuomy:pr-36

Conversation

@NonaSuomy
Copy link
Copy Markdown

added directories_config.bat

Comment thread SendKeys_Accept.vbs
Comment thread 00_Download_Script.bat
@NonaSuomy
Copy link
Copy Markdown
Author

NonaSuomy commented Jul 3, 2025

Any issues with this now?
Changed all the files to lf instead of crlf which seems strange as it's windows scripting not unix.

@NonaSuomy NonaSuomy changed the title Pr 36 Add customized directories issues/21 Jul 3, 2025
@NonaSuomy NonaSuomy requested a review from FREEWING-JP July 4, 2025 05:04
@NonaSuomy
Copy link
Copy Markdown
Author

Is there anything else you want done or good 2 go?

@pizzapi2012
Copy link
Copy Markdown
Contributor

@FREEWING-JP ?

@FREEWING-JP
Copy link
Copy Markdown
Owner

I will check contents .

Copy link
Copy Markdown
Owner

@FREEWING-JP FREEWING-JP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First , I pointed out the parts that concerned me .

Comment thread 00_Execute_Script.bat
echo %0
cd /d \00_fritzing
rem Load configuration
if exist directories_config.bat call directories_config.bat
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if exist directories_config.bat call directories_config.bat
cange to
call directories_config.bat

The directories_config.bat file always exists, so there is no need to check for its existence .
Of course , this also applies to batch files 01 to 20 .

Comment thread 00_Download_Script.bat
Comment on lines +8 to +38
rem Load configuration from directories_config.bat if it exists
rem Otherwise, use default values
if exist directories_config.bat (
echo Loading configuration from directories_config.bat
call directories_config.bat
) else (
echo directories_config.bat not found, creating with default values
rem Configure Qt installation directory (default is C:\Qt)
if "%QT_INSTALL_DIR%" == "" set QT_INSTALL_DIR=C:\Qt

rem Configure Fritzing workspace directory (default is \00_fritzing)
if "%FRITZING_WORKSPACE_DIR%" == "" set FRITZING_WORKSPACE_DIR=\00_fritzing

rem Configure Visual Studio installation directory
if "%VS_INSTALL_DIR%" == "" set VS_INSTALL_DIR=C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools

rem Create the configuration file
echo @echo off> directories_config.bat
echo rem directories_config.bat>> directories_config.bat
echo rem Configuration file for directory paths used in Fritzing build scripts>> directories_config.bat
echo rem This file is meant to be persistent and can be edited manually>> directories_config.bat
echo.>> directories_config.bat
echo rem Qt installation directory>> directories_config.bat
echo set QT_INSTALL_DIR=%QT_INSTALL_DIR%>> directories_config.bat
echo.>> directories_config.bat
echo rem Fritzing workspace directory>> directories_config.bat
echo set FRITZING_WORKSPACE_DIR=%FRITZING_WORKSPACE_DIR%>> directories_config.bat
echo.>> directories_config.bat
echo rem Visual Studio installation directory>> directories_config.bat
echo set VS_INSTALL_DIR=%VS_INSTALL_DIR%>> directories_config.bat
)
Copy link
Copy Markdown
Owner

@FREEWING-JP FREEWING-JP Sep 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rem Load configuration from directories_config.bat if it exists
rem Otherwise, use default values
if exist directories_config.bat (
...
    echo set VS_INSTALL_DIR=%VS_INSTALL_DIR%>> directories_config.bat
)

change to
call directories_config.bat

Because Will change it to do the same processing in directories_config.bat , so can delete this section .

Comment thread 00_Download_Script.bat
Comment on lines +71 to +89
rem Update directories_config.bat if it already exists
if not exist directories_config.bat (
rem Create the configuration file
echo @echo off> directories_config.bat
echo rem directories_config.bat>> directories_config.bat
echo rem Configuration file for directory paths used in Fritzing build scripts>> directories_config.bat
echo rem This file is meant to be persistent and can be edited manually>> directories_config.bat
echo.>> directories_config.bat
echo rem Qt installation directory>> directories_config.bat
echo set QT_INSTALL_DIR=%QT_INSTALL_DIR%>> directories_config.bat
echo.>> directories_config.bat
echo rem Fritzing workspace directory>> directories_config.bat
echo set FRITZING_WORKSPACE_DIR=%FRITZING_WORKSPACE_DIR%>> directories_config.bat
echo.>> directories_config.bat
echo rem Visual Studio installation directory>> directories_config.bat
echo set VS_INSTALL_DIR=%VS_INSTALL_DIR%>> directories_config.bat
echo.>> directories_config.bat
echo rem Created by 00_Download_Script.bat>> directories_config.bat
)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this section is not need .
Because directories_config.bat file always exists .

Comment thread directories_config.bat
Comment on lines +6 to +7
rem Qt installation directory (default is C:\Qt)
set QT_INSTALL_DIR=C:\Qt
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rem Qt installation directory (default is C:\Qt)
set QT_INSTALL_DIR=C:\Qt

change to

    rem Configure Qt installation directory (default is C:\Qt)
    if "%QT_INSTALL_DIR%" == "" set QT_INSTALL_DIR=C:\Qt

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FRITZING_WORKSPACE_DIR
VS_INSTALL_DIR

Change these as well .

Comment thread directories_config.bat
set VS_INSTALL_DIR=C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools

rem Additional configuration variables can be added here

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add

echo Using Qt installation directory: %QT_INSTALL_DIR%
echo Using Fritzing workspace directory: %FRITZING_WORKSPACE_DIR%
echo Using Visual Studio directory: %VS_INSTALL_DIR%

Comment on lines -86 to +88

pause No newline at end of file
Copy link
Copy Markdown
Owner

@FREEWING-JP FREEWING-JP Sep 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is abnormal for differences to occur in this location .
Please return to a state where there are no differences .
(When there are strange differences like this , I lose the desire to review .)

image

@FREEWING-JP
Copy link
Copy Markdown
Owner

image

I think this should be done in one commit .
Commits like this that go against my style make me less willing to review them .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants