Skip to content
This repository has been archived by the owner on Mar 26, 2024. It is now read-only.

Add possibility of having project files in different locations than root, when packaging webos project #115

Closed
wants to merge 1 commit into from

Conversation

ejbp
Copy link

@ejbp ejbp commented Feb 7, 2018

Add possibility of copying subdirectories into our project

ex [package.json]:
"react-tv": {
"files": [
"./public/index.html",
"./dist/bundle.js",
"./dist/assets/"
]
}

./public/index.html file will be copied to [webos_project_root]/index.html
./dist/bundle.js file will be copied to [webos_project_root]/bundle.js
./dist/assets/ dir and it's content will be copied to [webos_project_root]/assets

note 1) in this change, .npmignore was not implemented

note 2) If you add a directory that stand on the root, it will not work. ex:

"react-tv": {
"files": [
"./public/index.html",
"./dist/bundle.js",
"./dist/assets/",
"./videos/"
]
}

This was not implemented cause may cause corruption of our default webos_project files, like appinfo.json or even the [].ipk file itself. Problem it could be implemented when the .npmignore.

Add possibility of copying subdirectories into our project

ex [package.json]:
"react-tv": {
    "files": [
      "./public/index.html",
      "./dist/bundle.js",
      "./dist/assets/"
    ]
  }

./public/index.html file will be copied to [webos_project_root]/index.html
./dist/bundle.js file will be copied to [webos_project_root]/bundle.js
./dist/assets/ dir and it's content will be copied to [webos_project_root]/assets

note 1) in this change, .npmignore was not implemented

note 2) If you add a directory that stand on the root, it will not work. ex:

"react-tv": {
    "files": [
      "./public/index.html",
      "./dist/bundle.js",
      "./dist/assets/",
      "./videos/"
    ]
  }

This was not implemented cause may cause corruption of our default webos_project files, like appinfo.json or even the [].ipk file itself. Problem it could be implemented when the .npmignore.
@raphamorim
Copy link
Owner

Hi @ejbp, I don't know if this is the best approach for solving it.
Related issue to #110 #111

@raphamorim raphamorim closed this Apr 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants