New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error filtering resources: Mark invalid #254

Closed
ceeliro opened this Issue Jan 20, 2015 · 10 comments

Comments

Projects
None yet
7 participants
@ceeliro

ceeliro commented Jan 20, 2015

[INFO] Copying 10 resources to ../share-amp
...
[DEBUG] filtering /home/ceeliro/ICMR/workspace_wks_p1/test-alf-all-in-one/share-amp/src/main/amp/web/js/example/widgets/css/TemplateWidget.css to /home/ceeliro/ICMR/workspace_wks_p1/test-alf-all-in-one/share-amp/target/classes/../share-amp/web/js/example/widgets/css/TemplateWidget.css
...
Caused by: java.io.IOException: Mark invalid
at java.io.BufferedReader.reset(BufferedReader.java:505)
at org.apache.maven.shared.filtering.MultiDelimiterInterpolatorFilterReaderLineEnding.read(MultiDelimiterInterpolatorFilterReaderLineEnding.java:416)
at org.apache.maven.shared.filtering.MultiDelimiterInterpolatorFilterReaderLineEnding.read(MultiDelimiterInterpolatorFilterReaderLineEnding.java:205)
at java.io.Reader.read(Reader.java:140)
at org.apache.maven.shared.utils.io.IOUtil.copy(IOUtil.java:181)
at org.apache.maven.shared.utils.io.IOUtil.copy(IOUtil.java:168)
at org.apache.maven.shared.utils.io.FileUtils.copyFile(FileUtils.java:1856)
at org.apache.maven.shared.utils.io.FileUtils.copyFile(FileUtils.java:1804)
at org.apache.maven.shared.filtering.DefaultMavenFileFilter.copyFile(DefaultMavenFileFilter.java:114)

@ceeliro

This comment has been minimized.

Show comment
Hide comment
@ceeliro

ceeliro Jan 20, 2015

I modified the out of the box TemplateWidget.css file, in a completely new all-in-one archetype project, to add this line at the beginning

"@media (max-width: 768px) {...}"

This error also happens to me with many other files after upgrading the all-in-one archetype project that i am currently working with from SDK 2.0.0-beta4 to 2.0.0

ceeliro commented Jan 20, 2015

I modified the out of the box TemplateWidget.css file, in a completely new all-in-one archetype project, to add this line at the beginning

"@media (max-width: 768px) {...}"

This error also happens to me with many other files after upgrading the all-in-one archetype project that i am currently working with from SDK 2.0.0-beta4 to 2.0.0

@ModestasM

This comment has been minimized.

Show comment
Hide comment
@ModestasM

ModestasM Feb 7, 2015

I have this problem too with alfresco sdk version 2.0.0
The problem can be reproduced with fabric.js file taken from: http://fabricjs.com/

ModestasM commented Feb 7, 2015

I have this problem too with alfresco sdk version 2.0.0
The problem can be reproduced with fabric.js file taken from: http://fabricjs.com/

@ceeliro

This comment has been minimized.

Show comment
Hide comment
@ceeliro

ceeliro Feb 13, 2015

No clue in this regard? I have to skip filtering most of the files to be able to run the project (mostly min.js, js and css)

Thanks in advance

ceeliro commented Feb 13, 2015

No clue in this regard? I have to skip filtering most of the files to be able to run the project (mostly min.js, js and css)

Thanks in advance

@ModestasM

This comment has been minimized.

Show comment
Hide comment
@ModestasM

ModestasM Feb 13, 2015

this helped me

<build>
    <plugins>
        <plugin>
            <artifactId>maven-resources-plugin</artifactId>
            <configuration>
                <nonFilteredFileExtensions>
                    <nonFilteredFileExtension>js</nonFilteredFileExtension>
                    <nonFilteredFileExtension>css</nonFilteredFileExtension>
                </nonFilteredFileExtensions>
            </configuration>
        </plugin>
    </plugins>
</build>

ModestasM commented Feb 13, 2015

this helped me

<build>
    <plugins>
        <plugin>
            <artifactId>maven-resources-plugin</artifactId>
            <configuration>
                <nonFilteredFileExtensions>
                    <nonFilteredFileExtension>js</nonFilteredFileExtension>
                    <nonFilteredFileExtension>css</nonFilteredFileExtension>
                </nonFilteredFileExtensions>
            </configuration>
        </plugin>
    </plugins>
</build>
@ceeliro

This comment has been minimized.

Show comment
Hide comment
@ceeliro

ceeliro Feb 13, 2015

sure, thanks. I wanted to know if it is a known bug, if they are going to deal with it for further releases or so. But never answered :(

Thanks for the reply

ceeliro commented Feb 13, 2015

sure, thanks. I wanted to know if it is a known bug, if they are going to deal with it for further releases or so. But never answered :(

Thanks for the reply

@douglascrp

This comment has been minimized.

Show comment
Hide comment
@douglascrp

douglascrp Feb 13, 2015

Contributor

I usually include ico on that list.

Em Fri Feb 13 2015 at 7:37:56 AM, ceeliro notifications@github.com
escreveu:

sure, thanks. I wanted to know if it is a known bug, if they are going to
deal with it for further releases or so. But never answered :(

Thanks for the reply


Reply to this email directly or view it on GitHub
#254 (comment)
.

Contributor

douglascrp commented Feb 13, 2015

I usually include ico on that list.

Em Fri Feb 13 2015 at 7:37:56 AM, ceeliro notifications@github.com
escreveu:

sure, thanks. I wanted to know if it is a known bug, if they are going to
deal with it for further releases or so. But never answered :(

Thanks for the reply


Reply to this email directly or view it on GitHub
#254 (comment)
.

@mindthegab mindthegab added this to the 2.0.1 milestone Apr 2, 2015

@mindthegab mindthegab self-assigned this Apr 2, 2015

@kmezyaoui

This comment has been minimized.

Show comment
Hide comment
@kmezyaoui

kmezyaoui May 13, 2015

Hi,
Another file extension should be added to the list of ignored filter extensions (used in themes):

psd

regards.

kmezyaoui commented May 13, 2015

Hi,
Another file extension should be added to the list of ignored filter extensions (used in themes):

psd

regards.

@mindthegab mindthegab modified the milestones: 2.1.1, 2.0.1 May 27, 2015

@mindthegab

This comment has been minimized.

Show comment
Hide comment
@mindthegab

mindthegab May 27, 2015

Contributor

Added js, ico and psd to nonFilteredExtensions

Contributor

mindthegab commented May 27, 2015

Added js, ico and psd to nonFilteredExtensions

@mindthegab mindthegab closed this May 27, 2015

mindthegab added a commit that referenced this issue May 29, 2015

@mrgrechkinn

This comment has been minimized.

Show comment
Hide comment
@mrgrechkinn

mrgrechkinn Jan 22, 2016

mindthegab commented "Added js, ico and psd to nonFilteredExtensions"

I do not see where you have added js to nonFilteredExtension

mrgrechkinn commented Jan 22, 2016

mindthegab commented "Added js, ico and psd to nonFilteredExtensions"

I do not see where you have added js to nonFilteredExtension

@ohej ohej modified the milestones: 2.2.0, 2.1.1 Jan 22, 2016

@ohej ohej assigned ohej and unassigned mindthegab Jan 22, 2016

@ohej

This comment has been minimized.

Show comment
Hide comment
@ohej

ohej Jan 22, 2016

Collaborator

I see that we missed JS to the list. I've just committed it for 2.2, reopening and closing with the correct milestone so it shows up in the GitHub report when we release

Collaborator

ohej commented Jan 22, 2016

I see that we missed JS to the list. I've just committed it for 2.2, reopening and closing with the correct milestone so it shows up in the GitHub report when we release

@ohej ohej reopened this Jan 22, 2016

@ohej ohej closed this Jan 22, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment