simbit18 commented on code in PR #9782: URL: https://github.com/apache/nuttx/pull/9782#discussion_r2068184260
########## tools/nxmake2cmake.py: ########## @@ -0,0 +1,241 @@ +#!/usr/bin/env python3 +############################################################################ +# tools/nxmake2cmake.py +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distributed with +# this work for additional information regarding copyright ownership. The +# ASF licenses this file to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance with the +# License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT +# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the +# License for the specific language governing permissions and limitations +# under the License. +# +############################################################################ + +import argparse +import os +import sys +from string import Template + +if sys.version_info[0] < 3: + raise Exception("Must be using Python 3") + + +def board_cmakelist_get(path): + template = """# ############################################################################## +# $board_path/CMakeLists.txt +# +# SPDX-License-Identifier: Apache-2.0 +# +# Licensed to the Apache Software Foundation (ASF) under one or more contributor +# license agreements. See the NOTICE file distributed with this work for +# additional information regarding copyright ownership. The ASF licenses this +# file to you under the Apache License, Version 2.0 (the "License"); you may not +# use this file except in compliance with the License. You may obtain a copy of +# the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT +# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the +# License for the specific language governing permissions and limitations under +# the License. +# +# ############################################################################## + +add_subdirectory(src) +""" + temp_obj = Template(template) + return temp_obj.substitute(board_path=path) + + +def add_board_cmakelist(path): + cmpath = os.path.join(path, "CMakeLists.txt") + + print("Add", cmpath) + + with open(cmpath, "w") as f: + content = board_cmakelist_get(path) + content = board_cmakelist_get(path) + f.write(content) + + +def file_update_path(s, oldpath, newpath): + return s.replace(oldpath, newpath) + + +def convert_make_conditional(content): + content = content.replace(",y)", "") + content = content.replace("ifeq ($", "if") + content = content.replace("ifneq ($(", "if(NOT ") + content = content.replace("else", "else()") + content = content.replace("endif", "endif()") + content = content.replace("CHIP_CSRCS = ", "set(SRCS ") + content = content.replace("CHIP_CSRCS += ", " list(APPEND SRCS ") + content = content.replace("CSRCS = ", "set(SRCS ") + content = content.replace("CSRCS += ", " list(APPEND SRCS ") + content = content.replace(".c\n", ".c)\n") + + return content + + +# common for board Makefile and Make.defs +def convert_board_common(path, content): + content = convert_make_conditional(content) + + # remove make specific lines + content = "\n".join(line for line in content.split("\n") if "include" not in line) + content = "\n".join(line for line in content.split("\n") if "ASRCS" not in line) + content = "\n".join(line for line in content.split("\n") if "DEPPATH" not in line) + content = "\n".join(line for line in content.split("\n") if "VPATH" not in line) + content = "\n".join(line for line in content.split("\n") if "CFLAGS" not in line) + + content += "target_sources(board PRIVATE ${SRCS})\n\n" + + # linker script + if os.path.exists(os.path.join(path, "scripts/ld.script")): + content += 'set_property(GLOBAL PROPERTY LD_SCRIPT "${NUTTX_BOARD_DIR}/scripts/ld.script")' + elif os.path.exists(os.path.join(path, "scripts/flash.ld")): + content += 'set_property(GLOBAL PROPERTY LD_SCRIPT "${NUTTX_BOARD_DIR}/scripts/flash.ld")' + else: + print("ERROR: not found linker script") + + content += "\n" + content = content.replace("\n\n\n", "\n\n") + content = content.replace("SRCS ", "SRCS ") + + return content + + +# convert board src/Make.defs to CMakeLists.txt +def convert_board_makedefs(path): + mdpath = os.path.join(path, "src/Make.defs") + cpath = os.path.join(path, "src/CMakeLists.txt") + + content = "" + with open(mdpath, "r") as fm: + content = fm.read() + content = content.replace(mdpath, cpath) + content = convert_board_common(path, content) + + if content: + print("add", cpath) + with open(cpath, "w") as fc: + fc.write(content) + + +# convert board src/Makefile to CMakeLists.txt +def convert_board_makefile(path): + mpath = os.path.join(path, "src/Makefile") + cpath = os.path.join(path, "src/CMakeLists.txt") + + content = "" + with open(mpath, "r") as fm: + content = fm.read() + content = content.replace(mpath, cpath) Review Comment: On Windows, it does not replace the filename (Windows paths are different from Unix) simple change ` content = content.replace(mpath, cpath) ` with ` content = content.replace("src/Make.defs", "src/CMakeLists.txt")` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
