FileSystemNotFoundException when building #736

Closed
iBotPeaches opened this Issue Mar 18, 2015 · 4 comments

Comments

Projects
None yet
1 participant
@iBotPeaches
Owner

iBotPeaches commented Mar 18, 2015

Original issue 626 created by dan.gulkis on 2014-04-24T23:36:50.000Z:

What steps will reproduce the problem?

  1. ./apktool d Pound#Test.apk
  2. ./apktool b Pound#Test
  3. Must use an apt that has unknown files/dirs ("Copying unknown files/dir...")

What is the expected output? What do you see instead?

This should work fine. Instead it gets an exception. The problem seems to be that the filename was converted in the unknown files/dir case, so now the '#' has been changed to '%23' which causes a path that isn't found.

I: Using Apktool 2.0.0-Beta9 on Pound#Test
I: Checking whether sources has changed...
I: Checking whether resources has changed...
I: Building apk file...
I: Copying unknown files/dir...
Exception in thread "main" java.nio.file.FileSystemNotFoundException: /Users/dang/Downloads/apktool2/Pound%23Test/dist/Pound%23Test.apk
at com.sun.nio.zipfs.ZipFileSystem.<init>(ZipFileSystem.java:120)
at com.sun.nio.zipfs.ZipFileSystemProvider.newFileSystem(ZipFileSystemProvider.java:117)
at java.nio.file.FileSystems.newFileSystem(FileSystems.java:322)
at java.nio.file.FileSystems.newFileSystem(FileSystems.java:272)
at brut.androlib.Androlib.insertFolder(Androlib.java:599)
at brut.androlib.Androlib.buildUnknownFiles(Androlib.java:566)
at brut.androlib.Androlib.build(Androlib.java:294)
at brut.androlib.Androlib.build(Androlib.java:258)
at brut.apktool.Main.cmdBuild(Main.java:236)
at brut.apktool.Main.main(Main.java:88)

What version of the product are you using? On what operating system?

2.0.0-Beta9 on OSX Mavericks

Please provide any additional information below.

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Mar 18, 2015

Owner

Comment #1 originally posted by connor.tumbleson on 2014-04-25T15:12:44.000Z:

People using strange characters -_-

Owner

iBotPeaches commented Mar 18, 2015

Comment #1 originally posted by connor.tumbleson on 2014-04-25T15:12:44.000Z:

People using strange characters -_-

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Mar 18, 2015

Owner

Comment #2 originally posted by connor.tumbleson on 2014-04-30T14:02:57.000Z:

Okay, I fixed using Paths instead of URIs

The only reason we moved to URIs though was to fix some Chinese problems. Pushing them to patches to confirm before committing.

Owner

iBotPeaches commented Mar 18, 2015

Comment #2 originally posted by connor.tumbleson on 2014-04-30T14:02:57.000Z:

Okay, I fixed using Paths instead of URIs

The only reason we moved to URIs though was to fix some Chinese problems. Pushing them to patches to confirm before committing.

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Mar 18, 2015

Owner

Comment #3 originally posted by connor.tumbleson on 2014-05-01T12:42:11.000Z:

Guess I forgot to attach the patch.

Owner

iBotPeaches commented Mar 18, 2015

Comment #3 originally posted by connor.tumbleson on 2014-05-01T12:42:11.000Z:

Guess I forgot to attach the patch.

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Mar 18, 2015

Owner

Comment #4 originally posted by connor.tumbleson on 2014-05-03T12:44:10.000Z:

a031787

Owner

iBotPeaches commented Mar 18, 2015

Comment #4 originally posted by connor.tumbleson on 2014-05-03T12:44:10.000Z:

a031787

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