grunt uglify is missing files

[Théodore Biadala]

I'm looking at upgrading Drupal version of jQuery UI but since 1.12 and the folder change of some effects and widgets, the uglify tasks is missing a lot of files, everything that is in widgets/* and effects/* essentially. We're using the individual files so this is preventing up from upgrading (for reference: ​https://www.drupal.org/node/2533498)

$ grunt uglify
Running "uglify:main" (uglify) task
>> Destination dist/jquery-ui.min.js not written because src files were empty.
>> No files created.

Running "uglify:i18n" (uglify) task
>> Destination dist/i18n/jquery-ui-i18n.min.js not written because src files were empty.
>> No files created.

Running "uglify:ui/core.js" (uglify) task
>> 1 file created.

Running "uglify:ui/widget.js" (uglify) task
>> 1 file created.

Running "uglify:ui/mouse.js" (uglify) task
>> Destination dist/minified/mouse.js not written because src files were empty.
>> No files created.

Running "uglify:ui/draggable.js" (uglify) task
>> Destination dist/minified/draggable.js not written because src files were empty.
>> No files created.

Running "uglify:ui/droppable.js" (uglify) task
>> Destination dist/minified/droppable.js not written because src files were empty.
>> No files created.

Running "uglify:ui/resizable.js" (uglify) task
>> Destination dist/minified/resizable.js not written because src files were empty.
>> No files created.

Running "uglify:ui/selectable.js" (uglify) task
>> Destination dist/minified/selectable.js not written because src files were empty.
>> No files created.

Running "uglify:ui/sortable.js" (uglify) task
>> Destination dist/minified/sortable.js not written because src files were empty.
>> No files created.

By tweaking the Gruntfile.js I was able to get the rest of it by adding this to the minify object (but then there are leftover js files in dist) :

dynamic_mappings: {
	files: [
		{
			expand: true,
			cwd: 'ui/',
			src: ['**/*.js'],
			dest: 'dist/minified',
			ext: '.min.js'
		}
	],
	options: {
		banner: createBanner()
	}
}

[Scott González]

PR: ​https://github.com/jquery/jquery-ui/pull/1751

You should know that this is not an officially supported way to handle a build though. This task can go away at any time without warning. Why are you not using source files and minifying yourself?

[Théodore Biadala]

I didn't know it wasn't supported, nothing was said so since it was there I assumed it was supported.

We don't have any js tool in drupal core everything is manual. We should probably have something but we didn't need it so far. Now that I know we can plan for it. (even if it's just taking the current build file and maintaining it ourselves)

Thanks!

[Scott González]

Replying to theodoreb:

I didn't know it wasn't supported, nothing was said so since it was there I assumed it was supported.

In general, all code that does not have documentation is not supported.

[Théodore Biadala]

Good to know, it's the other way around for Drupal :þ

[Scott González]

In c218bee:

Build: Fix list of source files

Fixes #15052
Closes gh-1751