Topic: PEBakery Test Build  (Read 40090 times)

Re: PEBakery Test Build
« Reply #60 on: November 24, 2017, 08:27:23 AM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

*
Can you replace naming on log:
Fixed -> Builder
ex:
Fixed    EngineVersion    091
->
Builder    EngineVersion    091

Global -> Permanent
ex:
Global    Tools    %BaseDir%\Projects\Tools\Gena
->
Permanent    Tools    %BaseDir%\Projects\Tools\Gena







******
Reasons behind:
*
There can not be fixed variable for a plugin

*
Global variables is something else we use on some plugins.
ex: Macro Library
script.project :
If,ExistFile,%api%,AddVariables,%api%,ApiVar,GLOBAL

--> Macro Library [ApiVar] section becomes Global Variables for project

ps:
Global Variable on projects is very very rare
so far Macro Library use AND some plugins (1 or 2) on Gena I guess

I had removed Global Variables from plugins long time ago only 1 Macro Library + 1 or 2 extra remained.

Global Variable defination:
They are variables available on all project
Different from Permanent Variables
1) they are not written to script.project
2) they can be added during build and available on memory (Global)
3) when builder close Global Variables deleted

ps: that is the reason we have
script.project
Code: [Select]
[Process]
If,ExistFile,%api%,AddVariables,%api%,ApiVar,GLOBAL
on all projects.

As a result Macro Library commands available on all plugins.


**
Summary of Variables (and Macros) :
- Builder Variables
 (ex: %BaseDir%)
- Permanent Variables
- Global Variables
- Special Builder Variables
(eX: %ExitCode%)

Plugin Starts with above variables +:
- Plugin Variables
(ex: %ScriptDir%)
- [Interface] Variables
- [Variables]

- Parameters ( #1 #2 ...#9 )

ps: Just an idea, better handle them on different memory arrangements with the priority order.  :wink:

:turtle:

Re: PEBakery Test Build
« Reply #61 on: November 24, 2017, 05:50:41 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

%UserName% deprecated now from all projects,
following plugin updated:

Apps\Network\Flash Add

(in fact %UserName% left only at Flash Add )





To avoid confusions these plugins also updated:

Win7PESE/Win8PESE\Tweaks\Classic Shell
Win10ESE/Win8PESE/Win8.1SE/Yomi\Tweaks\My Computer - Name
Gena\Components\Tweaks Explorer\My Computer - Name
Gena\Finals\1 Optimizations\BootText
Downloads\AppYGS\CDVD\UltraISO
Downloads\AppYGS\HD Tasks\Recover\Partition Find and Mount

(Confusion: %UserName% variable used on these plugins not from builder, now renamed )

==>
no need %UserName% on PEBakery anymore. :great:


:turtle:

Re: PEBakery Test Build
« Reply #62 on: November 24, 2017, 05:59:14 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

%UserProfile% variable not exists on any plugin

==>

no need %UserProfile% on PEBakery anymore. :great:

:turtle:

Re: PEBakery Test Build
« Reply #63 on: November 24, 2017, 06:09:21 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

%TempDir% variable not exists on any plugin

==>

no need %TempDir% on PEBakery anymore. :great:

:turtle:

Re: PEBakery Test Build
« Reply #64 on: November 24, 2017, 06:26:11 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

%WindowsDir% variable not exists on any plugin

==>

no need %WindowsDir% on PEBakery anymore. :great:

:turtle:

Re: PEBakery Test Build
« Reply #65 on: November 26, 2017, 08:14:42 AM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

Macro Library updated v43.r93

:turtle:

Re: PEBakery Test Build
« Reply #66 on: November 26, 2017, 03:21:04 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

Following Reply 49
http://theoven.org/index.php?topic=2271.msg24741#msg24741

All projects now have "Config Source Folder" plugin Selected=True Mandatory=True

-->
no need PEBakery - Settings - Projects - Project Path Setting:
Source Directory
Target Directory
ISO File

+
All SE Main Plugins (script.project) updated with new instruction and picture to use "Config Source Folder"

:turtle:

Re: PEBakery Test Build
« Reply #67 on: November 26, 2017, 03:32:22 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

PEBakery incompatibility with FileBox:

Giving example with "Config Source Folder" :

1)
SF_FileBox=,1,13,31,104,408,20,dir
->
When I select source folder using button near SF_FileBox , source folder do not end with \
ex:
SF_FileBox=D:\S\Win10_x64_4,1,13,31,104,408,20,
-->
It should be:
SF_FileBox=D:\S\Win10_x64_4\,1,13,31,104,408,20,dir

2)
After selecting Source Folder
PEBakery deletes dir
ex:
SF_FileBox=D:\S\Win10_x64_4,1,13,31,104,408,20,
-->
It should be:
SF_FileBox=D:\S\Win10_x64_4\,1,13,31,104,408,20,dir




Summary:
SF_FileBox=D:\S\Win10_x64_4,1,13,31,104,408,20,
should be:
SF_FileBox=D:\S\Win10_x64_4\,1,13,31,104,408,20,dir



---->
Knowing how we use FileBox on plugins, above should not effect Build.
Still they are incompatibility with FileBox  :wink:



:turtle:

Re: PEBakery Test Build
« Reply #68 on: November 26, 2017, 08:34:24 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

another intesresting thing with FileBox:

choosing file with filebox gets file uppercase to FILE

ex:
Utils\PC Packed
default:
File_FileBox=D:\Users\Cemal\Desktop\FS_Capture.7z,1,13,40,76,404,30,file

selecting "Test - Me.cmd" at desktop it changes to
File_FileBox="D:\Users\Cemal\Desktop\Test - Me.cmd",1,13,40,76,404,30,FILE

I do not know why PEBakery changes file to FILE, such thing required ?

:turtle:

Re: PEBakery Test Build
« Reply #69 on: November 26, 2017, 09:25:51 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

About %CDDrive% - "Circular Reference" - IniRead with escape :

keep in mind SE also have:
[Variables]
%SystemDrive%=#$pSystemDrive#$p

on Gena I feel we will probably also add
[Variables]
%SystemDrive%=#$pSystemDrive#$p
for PEBakery in future ....

*
Keep in mind:
AddVariables, do not escape

*
Memo:
Maybe we can have an additional PEBakery command if required
ex:
Code: [Select]
IniReadEscapeNO,...
or
ex:
Code: [Select]
System,EscapeNO
IniRead,.......
ex:
Code: [Select]
System,EscapeNO,2
IniRead,.......
AddVariables,.....

So we can update mixed behaviour of wb on plugins when required.

not required for now, just a memo.  :wink: :thumbsup:




*
Further I will be waiting for next PEBakery to test things.
IniRead with escape
Set,Macro,Nil,Permanent
etc.

I feel there are enough things for PEBakery development till now.  :great:

See You
:turtle:

Re: PEBakery Test Build
« Reply #70 on: November 26, 2017, 09:38:57 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

Small thing:

"Refresh Plugin" button do not refresh
[Main]
Selected=
Mandatory=
Level=


:turtle:

Edit:
Continue with Reply 81
http://theoven.org/index.php?topic=2271.msg24832#msg24832
« Last Edit: December 02, 2017, 10:08:41 PM by Lancelot »

Re: PEBakery Test Build
« Reply #71 on: November 27, 2017, 07:01:42 AM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

Small thing:
A Setting required for "Edit" button behaviour
http://theoven.org/index.php?topic=2294

ex:
PEBakery - Setting - Editor:
"Edit Plugin Button"
Editor: %BaseDir%\Editor\NotepadPP\notepad++.exe

ps: If Editor not found back to default notepad.exe  :wink:

:turtle:

Re: PEBakery Test Build
« Reply #72 on: November 27, 2017, 05:16:18 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

IniWriteTextLine incompatibility :

Test Plugin:

Code: [Select]
[Process]
If,Not,ExistDir,%ProjectTemp%,DirMake,%ProjectTemp%
Set,%w%,%ProjectTemp%\Test.ini
Call,FileCreateBlank,%w%,OverWrite
IniWrite,%w%,Process,TEST,1
IniWriteTextLine,%w%,Process,"1"
IniWriteTextLine,%w%,Process,"2"
IniWriteTextLine,%w%,Process,"3"
IniWriteTextLine,%w%,Process,"4"
IniWriteTextLine,%w%,Process,"5"
IniWriteTextLine,%w%,Process,"6"
IniWriteTextLine,%w%,Process,"7"
IniWriteTextLine,%w%,Process,"8"
IniWriteTextLine,%w%,Process,"9"
Call,StartDoc,%w%

Result should be:

Code: [Select]
[Process]
9
8
7
6
5
4
3
2
1
TEST=1

PEBakery Result:

Code: [Select]
[Process]
2
3
4
5
6
7
8
9
1
TEST=1

 :wink:

:turtle:

Re: PEBakery Test Build
« Reply #73 on: December 02, 2017, 09:08:36 AM »

ChrisR

  • XPE Baker
  • Grand Chef
  • *****
  • Date Registered: Mar 2011
  • Posts: 3494
Hi ied206,
I just tested a standard build with PEBackery and all Lancelot's updates and everything worked fine  :thumbsup:
It's a really big job. Well done  :great:
Congratulations also for the open GPL3 license. The way to go. I hope you will have future contributions.
In other times, I would have tried to participate.
I hope it can become really better than just a fixed WB Clone. It deserves. Good luck
Thanks  :great:

Re: PEBakery Test Build
« Reply #74 on: December 02, 2017, 09:50:27 AM »

ied206

  • Moderator,
  • Chef
  • *****
  • Location: South Korea
  • Date Registered: Jan 2014
  • Posts: 257
Build 20171202 released.

EDIT
TestBuild taken down because of potential license issue.
Please use beta 2 or later, sorry for inconvenience.

1. Changelog
- Fix IniWriteTextLine prepend / append bug (caused by faulty optimization)
- Fix RadioButton clear bug
- IniRead will escape the read value.

2. Edit Button
Currently PEBakery just call ShellExecute api, opening the default editor for files with '.script' and '.project' extension.
Option to direct editor will be nice, it will be added on roadmap.

3. Empty Plugin is not refreshed
PEBakery ignores plugin which seems to be 'malformed', I think that is the case.
I will investigate about this.

4. Macro Manipulation
While I think controlling macro with variables is bad idea, we need compatibility for now.
Supporting Macro in Set command is work in progress.
Proposal : How about introducing SetMacro command, solely for macro manipulation?

However, PEBakery cannot support variable in macro name.
PEBakery internally parse macro command ahead of time for performance, and chaging it will cause massive architecture modification.

5. FileBox Behavior
5-1) file to FILE
PEBakery internally stores "FILE" flag as boolean, and write to file with fixed string "FILE".
It helps performance, because parsing is done only at first place.

5-2) DIR flag
I didn't know DIR flag exist, I will investigate.

6. Config Source Folder
I read this question rather too late, now it is already on the server :smile:.
I am curious about ChrisR's opinion by the way.

7. Stop Button Bug
Still investigating.

PS
Wow, lot of things were discovered. It would take time to catch up things...
« Last Edit: January 20, 2018, 07:45:04 PM by ied206 »

Re: PEBakery Test Build
« Reply #75 on: December 02, 2017, 09:53:12 AM »

ied206

  • Moderator,
  • Chef
  • *****
  • Location: South Korea
  • Date Registered: Jan 2014
  • Posts: 257
Hi ied206,
I just tested a standard build with PEBackery and all Lancelot's updates and everything worked fine  :thumbsup:
It's a really big job. Well done  :great:
Congratulations also for the open GPL3 license. The way to go. I hope you will have future contributions.
In other times, I would have tried to participate.
I hope it can become really better than just a fixed WB Clone. It deserves. Good luck
Thanks  :great:

Thanks, ChrisR!

Re: PEBakery Test Build
« Reply #76 on: December 02, 2017, 10:33:11 AM »

ChrisR

  • XPE Baker
  • Grand Chef
  • *****
  • Date Registered: Mar 2011
  • Posts: 3494
Thanks ied206,
Be free, really free, with Win10PESE, the parenthesis is now closed for me.
And so, I leave the hand to you, Lancelot, JFX or others in the directions that you think are the best.
I wish you all the best  :thumbsup:




Re: PEBakery Test Build
« Reply #77 on: December 02, 2017, 07:34:28 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ChrisR,

As I wrote on other topic,
Be around, Let us have your eye about PEBakery.

posts don't take much time.  :thumbsup:
And it takes quite less time building with PEBakery.  :xmas-thumbsup:
Come arond time to time to make a build test and comment.  :xmas-good:


**
I hope it can become really better than just a fixed WB Clone. It deserves. Good luck
I dream PEBakery would be 3rd generation builder following pebuilder,
and probably much better (There is already Auto Convert Plugins to UTF-8 option.....  :wink: )
And I feel after compatibility things solved, there will be development to next syntax ... Time will show ....

 :xmas-beer:
:turtle:

Re: PEBakery Test Build
« Reply #78 on: December 02, 2017, 08:53:38 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
Hi ied206,

I just update Gena Appetizer
http://gena.cwcodes.net/Compressed/index.php

PEBakery now builds Gena out of box.  :thumbsup:

RadioButton was a critical fix, now shortcuts working fine.  :thumbsup:

See you on next post... :turtle:

Re: PEBakery Test Build
« Reply #79 on: December 02, 2017, 09:14:18 PM »

Lancelot

  • Moderator, Gena Baker
  • Grand Chef
  • *****
  • Date Registered: Sep 2010
  • Posts: 10350
4. Macro Manipulation
While I think controlling macro with variables is bad idea, we need compatibility for now.
Supporting Macro in Set command is work in progress.
Proposal : How about introducing SetMacro command, solely for macro manipulation?

However, PEBakery cannot support variable in macro name.
PEBakery internally parse macro command ahead of time for performance, and chaging it will cause massive architecture modification.

SetMacro is good idea.  :thumbsup:

Since there are very small (but important) number of them inside plugins,
 it is ok to me to update them following PEBakery.  :great:

*
We will avoid using variable with SetMacro   :great:
I guess
Set,MenuAdd_GRUB,"Exec,%ScriptFile%,Add-grub4dos-entry",PERMANENT
will add script.project [Variables] something like:
Code: [Select]
[Variables]
...
MenuAdd_GRUB=Exec,"D:\Oven\Projects\Gena\Basic\5 BootsectorPE.script",Add-grub4dos-entry

which is fine  :great:

+
Let us know how to remove macros with SetMacro, following old way :
SetMacro,MenuAdd_GRUB,Nil,Permanent

*
I will implement SetMacro to plugins with next PEBakery  :xmas-thumbsup:

See you on next post...  :turtle:
« Last Edit: December 04, 2017, 10:51:55 AM by Lancelot »

 

Powered by EzPortal