Hi (I'm answering back here for the ML history).
I added this dependency back in r69160 because at that time, efisys depended on 
${CMAKE_BINARY_DIR}/boot/bootdata/BCD . But it was realized later (in r70626) 
that it was actually not needed; however the committer forgot to remote the 
bcd_hive dependency.

Best,
Hermès

-----Message d'origine-----
De : Ros-dev [mailto:ros-dev-boun...@reactos.org] De la part de Thomas Faber
Envoyé : samedi 13 mai 2017 19:20
À : Hermès BÉLUSCA-MAÏTO
Cc : ros-dev@reactos.org
Objet : Re: [ros-dev] [ros-diffs] [hbelusca] 69160: [EFISYS]: Add 'bcd_hive' as 
a dependency. [FATTEN]: Fix help message. Functions with no parameters are 
void. Remove an hardcoded array size.

Hey Hermès,
I'm fixing how the BCD hive is generated and this just showed up in an svn 
blame for me.
Can you explain why this bcd_hive dependency is needed? I don't see the command 
depending on the hive file.

Thanks,
Thomas


On 2015-09-09 21:48, hbelu...@svn.reactos.org wrote:
> Author: hbelusca
> Date: Wed Sep  9 19:48:15 2015
> New Revision: 69160
> 
> URL: http://svn.reactos.org/svn/reactos?rev=69160&view=rev
> Log:
> [EFISYS]: Add 'bcd_hive' as a dependency.
> [FATTEN]: Fix help message. Functions with no parameters are void. Remove an 
> hardcoded array size.
> 
> Modified:
>     trunk/reactos/boot/CMakeLists.txt
>     trunk/reactos/tools/fatten/fatten.c
> 
> Modified: trunk/reactos/boot/CMakeLists.txt
> URL: 
> http://svn.reactos.org/svn/reactos/trunk/reactos/boot/CMakeLists.txt?r
> ev=69160&r1=69159&r2=69160&view=diff
> ==============================================================================
> --- trunk/reactos/boot/CMakeLists.txt [iso-8859-1] (original)
> +++ trunk/reactos/boot/CMakeLists.txt [iso-8859-1] Wed Sep  9 19:48:15 2015
> @@ -17,7 +17,7 @@
>  
>  add_custom_target(efisys
>      COMMAND native-fatten ${CMAKE_CURRENT_BINARY_DIR}/efisys.bin -format 
> 2880 EFIBOOT -boot ${CMAKE_CURRENT_BINARY_DIR}/freeldr/bootsect/fat.bin 
> -mkdir EFI -mkdir EFI/BOOT -add $<TARGET_FILE:bootmgfw> 
> EFI/BOOT/boot${EFI_PLATFORM_ID}.efi -add 
> ${CMAKE_BINARY_DIR}/boot/bootdata/BCD EFI/BOOT/BCD
> -    DEPENDS native-fatten bootmgfw fat
> +    DEPENDS native-fatten fat bootmgfw bcd_hive
>      VERBATIM)
>  
>  ##bootcd
> 

_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev


_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to