Skip to content

Commit 824c5fb

Browse files
committed
Properly embed non-text files for IDF-based projects
Resolve platformio#639
1 parent e0e18e6 commit 824c5fb

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

builder/frameworks/_embed_files.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ def transform_to_asm(target, source, env):
125125
),
126126
suffix=".txt.o",
127127
),
128-
TxtToAsm=Builder(
128+
FileToAsm=Builder(
129129
action=env.VerboseAction(
130130
" ".join(
131131
[
@@ -136,7 +136,7 @@ def transform_to_asm(target, source, env):
136136
),
137137
"-DDATA_FILE=$SOURCE",
138138
"-DSOURCE_FILE=$TARGET",
139-
"-DFILE_TYPE=TEXT",
139+
"-DFILE_TYPE=$FILE_TYPE",
140140
"-P",
141141
join(
142142
env.PioPlatform().get_package_dir("framework-espidf") or "",
@@ -166,7 +166,13 @@ def transform_to_asm(target, source, env):
166166

167167
files = extract_files(flags, files_type)
168168
if "espidf" in env.subst("$PIOFRAMEWORK"):
169-
env.Requires(join("$BUILD_DIR", "${PROGNAME}.elf"), env.TxtToAsm(files))
169+
env.Requires(
170+
join("$BUILD_DIR", "${PROGNAME}.elf"),
171+
env.FileToAsm(
172+
files,
173+
FILE_TYPE="TEXT" if files_type == "embed_txtfiles" else "BINARY",
174+
),
175+
)
170176
else:
171177
embed_files(files, files_type)
172178
remove_config_define(flags, files_type)

0 commit comments

Comments
 (0)